-
Notifications
You must be signed in to change notification settings - Fork 4
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 rubocop to version '1.x' #33
Conversation
sidekiq-undertaker.gemspec
Outdated
@@ -25,6 +25,8 @@ Gem::Specification.new do |spec| | |||
"build_status_uri" => "https://travis-ci.org/ThomasKoppensteiner/sidekiq-undertaker" | |||
} | |||
|
|||
spec.required_ruby_version = ">= 2.4.0" |
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.
Related to rt_rubocop_defaults
. Try to upgrade it there first.
0e41e59
to
84026e4
Compare
.codeclimate.yml
Outdated
@@ -8,6 +8,6 @@ plugins: | |||
enabled: true | |||
rubocop: | |||
enabled: true | |||
channel: rubocop-0-77 | |||
channel: rubocop-1-2 |
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.
Needs codeclimate/codeclimate-rubocop#262 and puppetlabs/rubocop-i18n#46 to be merged and released
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.
Lets use 1.4
, when codeclimate/codeclimate-rubocop#263 is released
sidekiq-undertaker.gemspec
Outdated
spec.add_development_dependency "rt_rubocop_defaults", "~> 1.2" | ||
spec.add_development_dependency "rubocop", "~> 0.77" | ||
spec.add_development_dependency "rubocop-rspec", "~> 1.30" | ||
spec.add_development_dependency "rt_rubocop_defaults", "~> 2.1" |
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.
Wait until runtastic/rt_rubocop_defaults#13 is merged and released
ac22784
to
f182ffd
Compare
f182ffd
to
4329f70
Compare
Changes related to
rubocop
upgrade (#32).