-
Notifications
You must be signed in to change notification settings - Fork 16
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
add new shared rubocop config #163
Conversation
- "**/*.spec" | ||
TargetRubyVersion: 2.5 # 2.5 is the SLE15 version, so ensure that we use proper version | ||
# By default enable all new cops | ||
NewCops: enable |
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.
newly added. Basically default makes updating rubocop more iterative, but it does not fit to way how we update rubocop version.
Naming/VariableNumber: | ||
# do not check numbers usage in symbols as it often come outside of ruby | ||
# like `x86_64` | ||
CheckSymbols: false |
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.
new item
# "#{path}element" | ||
# so calls path.to_s and append element. Result is wrong path. | ||
Style/StringConcatenation: | ||
Enabled: false |
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.
new item, see explanation
# while there is agreement on reducing openstruct usage, | ||
# it is still used too widely, so disable only per module when | ||
# it is ready | ||
Style/OpenStructUse: |
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.
new item
|
||
# Skip warning for constant definition in tests | ||
# as in rspec we often locate testing data close to respective test | ||
Lint/ConstantDefinitionInBlock: |
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.
new item
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.
LGTM
✔️ Public Jenkins job #30 successfully finished |
✔️ Internal Jenkins job #19 successfully finished |
changes to previous config will be added as inline comment to this PR