Skip to content
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

Merged
merged 6 commits into from
Jul 18, 2024
Merged

Fix Style Cops #11067

merged 6 commits into from
Jul 18, 2024

Conversation

archanaserver
Copy link
Contributor

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

Copy link
Contributor

@qcjames53 qcjames53 left a 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?

@archanaserver
Copy link
Contributor Author

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?

@qcjames53 i just rebased it!

@ianballou
Copy link
Member

@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.
@archanaserver
Copy link
Contributor Author

@archanaserver looks like we need another rebase because I merged a bunch of your PRs :)

thanks @ianballou , and rebased it again!

@qcjames53 qcjames53 merged commit d295c9b into Katello:master Jul 18, 2024
26 of 28 checks passed
@archanaserver archanaserver deleted the fix-style-cops branch July 18, 2024 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants