[WIP] Automated request of codeowner for PR review

[europ]

[WIP]

This was necessary due to the fact that there is not possible to choose number zero for required approving reviews in the settings of branch protection. Codeowners are requested for a pull request review depending on specifications defined in CODEOWNERS file. This file includes files specifications and authors related to them. Files from the PR are compared if they match any of the pattern defined in the codeowners file. If there is match then a corresponding user is requested for a review.

[miq-bot]

Some comments on commit europ@a1a725e

app/workers/codeowner_review_request.rb

• 💣 💥 🔥 🚒 - 2 - Detected byebug. Remove all debugging statements.
• 💣 💥 🔥 🚒 - 20 - Detected binding.pry. Remove all debugging statements.

[miq-bot]

Checked commit europ@a1a725e with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 22 offenses detected

app/workers/codeowner_review_request.rb

• ⚠️ - Line 20, Col 5 - Lint/Debugger - Remove debugger entry point binding.pry.
• ❗ - Line 108, Col 24 - Style/DefWithParentheses - Omit the parentheses in defs when the method doesn't accept any arguments.
• ❗ - Line 110, Col 7 - Layout/AlignHash - Align the elements of a hash literal if they span more than one line.
• ❗ - Line 112, Col 7 - Layout/AlignHash - Align the elements of a hash literal if they span more than one line.
• ❗ - Line 119, Col 38 - Style/AndOr - Use || instead of or.
• ❗ - Line 134, Col 19 - Style/Not - Use ! instead of not.
• ❗ - Line 134, Col 5 - Style/NegatedIf - Favor unless over if for negative conditions.
• ❗ - Line 136, Col 18 - Layout/SpaceAroundOperators - Surrounding space missing for operator +.
• ❗ - Line 141, Col 26 - Layout/SpaceAroundOperators - Surrounding space missing for operator +.
• ❗ - Line 141, Col 31 - Layout/SpaceAroundOperators - Surrounding space missing for operator +.
• ❗ - Line 155, Col 17 - Layout/SpaceAroundOperators - Surrounding space missing for operator +.
• ❗ - Line 155, Col 24 - Layout/SpaceAroundOperators - Surrounding space missing for operator +.
• ❗ - Line 156, Col 7 - Style/UnlessElse - Do not use unless with else. Rewrite these with the positive case first.
• ❗ - Line 19, Col 1 - Layout/EmptyLinesAroundMethodBody - Extra empty line detected at method body beginning.
• ❗ - Line 22, Col 32 - Style/MethodCallWithoutArgsParentheses - Do not use parentheses for method calls with no arguments.
• ❗ - Line 24, Col 1 - Style/BlockComments - Do not use block comments.
• ❗ - Line 43, Col 7 - Style/CommentAnnotation - Annotation comment, with keyword TODO, is missing a note.
• ❗ - Line 56, Col 1 - Style/BlockComments - Do not use block comments.
• ❗ - Line 93, Col 26 - Style/InverseMethods - Use reject instead of inverting select.
• ❗ - Line 93, Col 51 - Layout/SpaceBeforeBlockBraces - Space missing to the left of {.
• ❗ - Line 97, Col 30 - Style/InverseMethods - Use reject instead of inverting select.
• ❗ - Line 97, Col 55 - Layout/SpaceBeforeBlockBraces - Space missing to the left of {.

[europ]

@skateman

GitHub has fixed this already. This pull request is useless from now ...