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
Update to rubocop 1.13.0 and re-enable Bundler/GemComment
.
#44
Changes from 4 commits
3fbf0cb
d97ee17
1d96fca
4708f67
bbd011e
75c1c64
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
@@ -1,3 +1,7 @@ | ||||||
require: | ||||||
- rubocop-performance | ||||||
- rubocop-rails | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why did we need this change? If we do need it, should the rubocop-rails require go in rubocop_rails.yml? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @jturkel The need for this is that otherwise we get these errors: As for moving salsify_rubocop/conf/rubocop_without_rspec.yml Lines 192 to 193 in 8dd2369
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I guess this gem depends on rubocop-rails even if the project isn't using Rails so this should be innocuous. |
||||||
|
||||||
AllCops: | ||||||
NewCops: disable | ||||||
DisplayCopNames: true | ||||||
|
@@ -10,10 +14,10 @@ AllCops: | |||||
- 'tmp/**/*' | ||||||
- 'vendor/**/*' | ||||||
|
||||||
# Don't check version_specifiers until https://github.com/rubocop/rubocop/pull/9358 merges | ||||||
Bundler/GemComment: | ||||||
Enabled: true | ||||||
OnlyFor: | ||||||
- 'restrictive_version_specificiers' | ||||||
- 'source' | ||||||
- 'git' | ||||||
- 'github' | ||||||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
# frozen_string_literal: true | ||
|
||
module SalsifyRubocop | ||
VERSION = '1.0.2' | ||
VERSION = '1.1.0' | ||
end |
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.
Should be bump this (and the minimum Ruby version in the gemspec) to Ruby 2.6 since Ruby 2.5 is EOL now?