-
Notifications
You must be signed in to change notification settings - Fork 293
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
Fix Style Cops #11067
Fix Style Cops #11067
Conversation
28a3c8c
to
04401be
Compare
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.
Hey Archana! Your PR looks great to me, I'm going to ACK. Quick question: It looks like there are some conflicts with the branch, would you like me to take care of those when we squash and merge or would you rather rebase and re-push first?
04401be
to
0046c56
Compare
@qcjames53 i just rebased it! |
@archanaserver looks like we need another rebase because I merged a bunch of your PRs :) |
This pull request addresses the Style/RedundantRegexpEscape RuboCop offense by removing unnecessary escapes in the regular expression pattern.
Disabled several Style cops which may cause failure after applying changes also some are unsafe to do so for huge number of offenses fix. Also followed other foreman rubocop rules.
0046c56
to
0c613d4
Compare
thanks @ianballou , and rebased it again! |
What are the changes introduced in this pull request?
#11009
This pull request addresses several style issues in the codebase and updates the .rubocop.yml configuration file to better manage RuboCop offenses.
Considerations taken when implementing this change?
Check commits for details
What are the testing steps for this pull request?
bundle exec rubocop