{"action":"created","issue":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/2960","labels_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/2960/labels{/name}","comments_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/2960/comments","events_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/2960/events","html_url":"https://github.com/ManageIQ/manageiq/pull/2960","id":78598756,"number":2960,"title":"Only use io-extra when supported","user":{"login":"djberg96","id":78529,"avatar_url":"https://avatars.githubusercontent.com/u/78529?v=3","gravatar_id":"","url":"https://api.github.com/users/djberg96","html_url":"https://github.com/djberg96","followers_url":"https://api.github.com/users/djberg96/followers","following_url":"https://api.github.com/users/djberg96/following{/other_user}","gists_url":"https://api.github.com/users/djberg96/gists{/gist_id}","starred_url":"https://api.github.com/users/djberg96/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/djberg96/subscriptions","organizations_url":"https://api.github.com/users/djberg96/orgs","repos_url":"https://api.github.com/users/djberg96/repos","events_url":"https://api.github.com/users/djberg96/events{/privacy}","received_events_url":"https://api.github.com/users/djberg96/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-05-20T16:40:05Z","updated_at":"2015-05-20T16:48:02Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ManageIQ/manageiq/pulls/2960","html_url":"https://github.com/ManageIQ/manageiq/pull/2960","diff_url":"https://github.com/ManageIQ/manageiq/pull/2960.diff","patch_url":"https://github.com/ManageIQ/manageiq/pull/2960.patch"},"body":"This patch modifies the RawBlockIO.rb file so that it only requires io-extra and uses directio on unix platforms. Also, it uses require_relative instead of mangling $LOAD_PATH.\r\n\r\nBasically, this means it won't try to use io-extra if you're using JRuby or on Windows."},"comment":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/comments/103956004","html_url":"https://github.com/ManageIQ/manageiq/pull/2960#issuecomment-103956004","issue_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/2960","id":103956004,"user":{"login":"miq-bot","id":7806257,"avatar_url":"https://avatars.githubusercontent.com/u/7806257?v=3","gravatar_id":"","url":"https://api.github.com/users/miq-bot","html_url":"https://github.com/miq-bot","followers_url":"https://api.github.com/users/miq-bot/followers","following_url":"https://api.github.com/users/miq-bot/following{/other_user}","gists_url":"https://api.github.com/users/miq-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/miq-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/miq-bot/subscriptions","organizations_url":"https://api.github.com/users/miq-bot/orgs","repos_url":"https://api.github.com/users/miq-bot/repos","events_url":"https://api.github.com/users/miq-bot/events{/privacy}","received_events_url":"https://api.github.com/users/miq-bot/received_events","type":"User","site_admin":false},"created_at":"2015-05-20T16:48:02Z","updated_at":"2015-05-20T16:48:02Z","body":"<rubocop />Checked commit https://github.com/djberg96/manageiq/commit/ba72385f2be78705ddd1de7b163cec4cf5df435c with rubocop 0.27.1\n1 file checked, 2 offenses detected\n\n**lib/disk/modules/RawBlockIO.rb**\n- [ ] :large_orange_diamond: - [Line 1](https://github.com/djberg96/manageiq/blob/ba72385f2be78705ddd1de7b163cec4cf5df435c/lib/disk/modules/RawBlockIO.rb#L1), Col 1 - [Style/FileName](http://rubydoc.info/gems/rubocop/0.27.1/RuboCop/Cop/Style/FileName) - Use snake_case for source file names.\n- [ ] :large_orange_diamond: - [Line 3](https://github.com/djberg96/manageiq/blob/ba72385f2be78705ddd1de7b163cec4cf5df435c/lib/disk/modules/RawBlockIO.rb#L3), Col 1 - [Style/IfUnlessModifier](http://rubydoc.info/gems/rubocop/0.27.1/RuboCop/Cop/Style/IfUnlessModifier) - Favor modifier `if` usage when having a single-line body. Another good alternative is the usage of control flow `&&`/`||`.\n"}}