{"push_id":609979662,"size":257,"distinct_size":257,"ref":"refs/heads/automate_granular_class_import","head":"1b6f29662eefcecd297a2b3a2dc8c84b80987234","before":"67d26ccf8e5a2c5e36130d9b66e8015db1684e8e","commits":[{"sha":"b076555d3621a698634ef81d451754a1d252c12f","author":{"email":"58c3c50a243ae1e5a60f7a187b83c22a5a980873@hotmail.com","name":"sgpal"},"message":"Update miq_report.rb\n\nLine 60 appears to have a bug that causes the Reports module to not load for ManageIQ. The SQL query should not have a Group By instead it should be a Order by.","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/b076555d3621a698634ef81d451754a1d252c12f"},{"sha":"c1d5d424073ddf3f056d1a24ed52f43296ae85cf","author":{"email":"4c3bf2294c1004f3fd31587824197e9c39c17ab4@redhat.com","name":"Alberto Bellotti"},"message":"[api] rspec refactoring - spec/requests/api/vms_spec.rb\n\n- Added url helper methods for generating URL's\n- Added an api_basic_authorize helper method set both role and basic_authorize in one shot.\n- New helper method to fetch values dynamically from shared_examples.\n- Adding shared examples via spec/requests/shared_examples/api.rb\n- Converted spec/requests/api/vms_spec.rb to using shared examples\n\nhttps://trello.com/c/p76x7lcu","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/c1d5d424073ddf3f056d1a24ed52f43296ae85cf"},{"sha":"63c39513e0115f1a1b08d1889f71dfda233c1112","author":{"email":"f6f0daf8d830eefd236ae8318afee373bfff1197@redhat.com","name":"Martin Hradil"},"message":"Split AssetTagImport and ClassificationImport from miq_bulk_import.rb into their own files\n\nPrevents autoloader from failing to load ClassificationImport when referred to in session\n\nhttps://bugzilla.redhat.com/show_bug.cgi?id=1198735","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/63c39513e0115f1a1b08d1889f71dfda233c1112"},{"sha":"17297cbfd6cd6f32a4c42c6e9414e1e7b6338ca8","author":{"email":"7bbcc45b0f8f88a262415b9271579a59b891be2e@gmail.com","name":"Aaron Patterson"},"message":"use collect(&:strip) rather than keeping an index","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/17297cbfd6cd6f32a4c42c6e9414e1e7b6338ca8"},{"sha":"47f3f0af73b86783c6fb6d452831eab3d6291a90","author":{"email":"7bbcc45b0f8f88a262415b9271579a59b891be2e@gmail.com","name":"Aaron Patterson"},"message":"use select / reject and assign to the variable\n\nwe don't need to use `push` in this instance since we're just building\nan array","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/47f3f0af73b86783c6fb6d452831eab3d6291a90"},{"sha":"b39e0c4ee9230dd8b0c8362332d8292c6d034bb7","author":{"email":"7bbcc45b0f8f88a262415b9271579a59b891be2e@gmail.com","name":"Aaron Patterson"},"message":"use `select` to build the array rather than `push`","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/b39e0c4ee9230dd8b0c8362332d8292c6d034bb7"},{"sha":"fadf73d7afab6cffe4ab344d0c25711400abea94","author":{"email":"7bbcc45b0f8f88a262415b9271579a59b891be2e@gmail.com","name":"Aaron Patterson"},"message":"allocate arrays with initial values rather than empty arrays","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/fadf73d7afab6cffe4ab344d0c25711400abea94"},{"sha":"8e39fb05fc26d27ba3bd32628ca72ee5c0d2ea08","author":{"email":"7bbcc45b0f8f88a262415b9271579a59b891be2e@gmail.com","name":"Aaron Patterson"},"message":"only call `get_reports_menu` once","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/8e39fb05fc26d27ba3bd32628ca72ee5c0d2ea08"},{"sha":"6d6fb1ea1f3845a2d0adab83f6c601604b8afbdd","author":{"email":"7bbcc45b0f8f88a262415b9271579a59b891be2e@gmail.com","name":"Aaron Patterson"},"message":"find all roles in one query, then map","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/6d6fb1ea1f3845a2d0adab83f6c601604b8afbdd"},{"sha":"c9f3ec83f72b73e3d0e6b9a53dae0ef0d8a658b4","author":{"email":"7bbcc45b0f8f88a262415b9271579a59b891be2e@gmail.com","name":"Aaron Patterson"},"message":"find all groups in one query","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/c9f3ec83f72b73e3d0e6b9a53dae0ef0d8a658b4"},{"sha":"a3047b52e00b78912b7576f6a8c5bff5deb09b90","author":{"email":"7bbcc45b0f8f88a262415b9271579a59b891be2e@gmail.com","name":"Aaron Patterson"},"message":"sorting empty lists doesn't matter","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/a3047b52e00b78912b7576f6a8c5bff5deb09b90"},{"sha":"01a0b8f803c581a57832f312f65f5b830791a6ca","author":{"email":"7bbcc45b0f8f88a262415b9271579a59b891be2e@gmail.com","name":"Aaron Patterson"},"message":"set defaults early so we don't have to ||=","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/01a0b8f803c581a57832f312f65f5b830791a6ca"},{"sha":"e95ea3419c668b31a4097a8e9fd9a1e8a49b002a","author":{"email":"7bbcc45b0f8f88a262415b9271579a59b891be2e@gmail.com","name":"Aaron Patterson"},"message":"use sort_by rather than sort","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/e95ea3419c668b31a4097a8e9fd9a1e8a49b002a"},{"sha":"cf3c62c769913154adef69b04eea5dfe89d5bac3","author":{"email":"f6f0daf8d830eefd236ae8318afee373bfff1197@redhat.com","name":"Martin Hradil"},"message":"_x_edit_buttons - supoort apply_method - Apply needs to POST for settings_import*\n\nThere are no GET routes for settings_import or settings_import_tags,\nthus the Apply button needs to POST in that case\n\nhttps://bugzilla.redhat.com/show_bug.cgi?id=1198735","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/cf3c62c769913154adef69b04eea5dfe89d5bac3"},{"sha":"7387b585c977f26ee1f3db3668071dacb62757bd","author":{"email":"ea442e001a5636eeee710090794bb7eb2e3459c8@redhat.com","name":"Joe Rafaniello"},"message":"Default to using the openstack hostname for event monitoring.\n\nRelated to #1936 and #1800\n\nWe originally defaulted to ipaddress with this code in 99d6c2dd8f6:\n\n```ruby\n  def event_monitor_available?\n    begin\n      require 'openstack/openstack_event_monitor'\n      options = {}\n      options[:username] = self.authentication_userid(:amqp)\n      options[:password] = self.authentication_password(:amqp)\n      options[:hostname] = self.ipaddress\n      OpenstackEventMonitor.available?(options)\n    rescue => e\n    end\n  end\n```","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/7387b585c977f26ee1f3db3668071dacb62757bd"},{"sha":"59da8fa724d6f98039e1bea433438fd12310849a","author":{"email":"f6f0daf8d830eefd236ae8318afee373bfff1197@redhat.com","name":"Martin Hradil"},"message":"ops_controller - style cleanup - [\"foo\",\"bar\"].include?x -> %w(foo bar).include?(x)","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/59da8fa724d6f98039e1bea433438fd12310849a"},{"sha":"400d540fd296bec6520d60c9f6bafa43527dc10c","author":{"email":"a980123d2017102d2a6fc06a82e18b253b66d874@redhat.com","name":"Harpreet Kataria"},"message":"Fixed value of type parameter being sent upto server\n\nFixed value of type parameter being sent upto server in the \"Remove this Retirement Entry Point\" link, this was preventing user from removing previously saved Retirement Entry point value while editing Catalog Item.","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/400d540fd296bec6520d60c9f6bafa43527dc10c"},{"sha":"4bf4206cdc9a907b773957b9772071cdc4220b04","author":{"email":"4c3bf2294c1004f3fd31587824197e9c39c17ab4@redhat.com","name":"Alberto Bellotti"},"message":"[api] Supporting /api/providers actions\n\n- Enabling POST & DELETE verbs\n- supporting provider \"create\" action\n- renaming \"add\" to \"create\" method, supporting non action based POSTS.\n- supporting provider \"edit\" action\n- supporting provider \"refresh\" action\n- supporting provider \"delete\" action in async mode","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/4bf4206cdc9a907b773957b9772071cdc4220b04"},{"sha":"3bbfef56ce6a0d6de48b441fb2cf5417b57e0990","author":{"email":"a980123d2017102d2a6fc06a82e18b253b66d874@redhat.com","name":"Harpreet Kataria"},"message":"No need to resizetaskbar on replace right cell.\n\nThis only needs to be done when resizing browser window or layouts panel to adjust taskbar depending upon div positioning. This was causing right cell header to move down in all explorers when right cell was replaced.","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/3bbfef56ce6a0d6de48b441fb2cf5417b57e0990"},{"sha":"fb2b9f2610acfc63bc54efc91a28bc8867264008","author":{"email":"7bbcc45b0f8f88a262415b9271579a59b891be2e@gmail.com","name":"Aaron Patterson"},"message":"use nil? instead of == nil","distinct":true,"url":"https://api.github.com/repos/eclarizio/manageiq/commits/fb2b9f2610acfc63bc54efc91a28bc8867264008"}]}