We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Performance Cops will be extracted since v0.68.0
But onkcop includes Performance cop setting and onkcop will implicitly depend on rubocop-performance https://github.com/onk/onkcop/blob/v0.53.0.3/config/rubocop.yml#L166-L171
onkcop
Performance
rubocop-performance
Which of the following do you like?
Add rubocop-performance to onkcop as runtime dependency
Remove Performance/Casecmp from config/rubocop.yml
Performance/Casecmp
config/rubocop.yml
Other
The text was updated successfully, but these errors were encountered:
I plan to extract performance cop settings into a separate file, just as config/rails.yml.
config/rails.yml
Sorry, something went wrong.
Extract config for rubocop-performance
3ac0a25
Closes #61
Successfully merging a pull request may close this issue.
Context
Performance Cops will be extracted since v0.68.0
But
onkcop
includesPerformance
cop setting andonkcop
will implicitly depend onrubocop-performance
https://github.com/onk/onkcop/blob/v0.53.0.3/config/rubocop.yml#L166-L171
Which of the following do you like?
Plan A
Add
rubocop-performance
toonkcop
as runtime dependencyPlan B
Remove
Performance/Casecmp
fromconfig/rubocop.yml
Plan C
Other
The text was updated successfully, but these errors were encountered: