Skip to content

Update Rubocop to latest version#11590

Merged
aduth merged 7 commits intomainfrom
aduth-rubocop-1-69
Dec 4, 2024
Merged

Update Rubocop to latest version#11590
aduth merged 7 commits intomainfrom
aduth-rubocop-1-69

Conversation

@aduth
Copy link
Copy Markdown
Contributor

@aduth aduth commented Dec 4, 2024

🛠 Summary of changes

Updates Rubocop and related dependencies to latest versions, resolving outstanding issues and enabling some of the newly-available cops.

Not all newly-available cops were enabled. I used some personal judgment based on severity, existing problems, and style to make a best judgment, but others' opinions are welcome.

Changelogs:

Part of the motivation here is to bring in my own upstream patch rubocop/rubocop#13431 to address the false negative now fixed in app/jobs/account_creation_threat_metrix_job.rb.

📜 Testing Plan

Verify build passes.

aduth added 6 commits December 4, 2024 08:38
changelog: Internal, Dependencies, Update dependencies to latest version
Avoid deprecation warnings:

"Inheriting from `RuboCop::Cop::Cop` is deprecated. Use `RuboCop::Cop::Base` instead."
@aduth aduth merged commit 64fc600 into main Dec 4, 2024
@aduth aduth deleted the aduth-rubocop-1-69 branch December 4, 2024 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants