-
-
Notifications
You must be signed in to change notification settings - Fork 269
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Compatibility with RuboCop v0.60.0 #710
Conversation
manual/cops_rails.md
Outdated
@@ -41,6 +41,7 @@ it { is_expected.to have_http_status :error } | |||
|
|||
Name | Default value | Configurable values | |||
--- | --- | --- | |||
VersionAdded | `0.54` | Float |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in #711
The point number |
Probably also related to rubocop/rubocop#5251 and rubocop/rubocop#3414. |
I disagree with the recent changes in AlignHash, so a combination of `key` and `table` styles is no longer allowed. The issue is very well described in rubocop/rubocop#6410 .
90d55ed
to
cb4984f
Compare
@Darhazer Fixed and rebased. |
There are a few interesting changes between RuboCop v0.59.2 and v0.60.0:
[:exclude_limit]
when checkingcop.config_to_allow_offenses
.key
andtable
styles is no longer allowed. I disagree with this decision, and have voiced my opinion on the issue Feature request: Layout/AlignHash option to allow forkey
andtable
styles to both be permissible rubocop#6410Rails/HttpStatus
andRSpec/Rails/HttpStatus
… again. @Darhazer would you take a look at my last commit “tmp”?Before submitting the PR make sure the following are checked:
master
(if not - rebase it).bundle exec rake
) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).