Skip to content

Rubocop config: Use plugins for rubocop-rails and rubocop-performance#233

Closed
mamhoff wants to merge 2 commits intosolidusio:mainfrom
mamhoff:rubocop-plugins
Closed

Rubocop config: Use plugins for rubocop-rails and rubocop-performance#233
mamhoff wants to merge 2 commits intosolidusio:mainfrom
mamhoff:rubocop-plugins

Conversation

@mamhoff
Copy link
Contributor

@mamhoff mamhoff commented May 2, 2025

Since Rubocop 1.72, we have to use plugins instead of require.

@mamhoff mamhoff force-pushed the rubocop-plugins branch from 1b594d8 to 9a57476 Compare May 2, 2025 12:14
Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lets lower the ruby req

mamhoff added 2 commits May 5, 2025 14:55
Since Rubocop 1.72, we have to use plugins instead of require.
2.5 is really, really old.
@mamhoff mamhoff force-pushed the rubocop-plugins branch from e089cc6 to 902e886 Compare May 5, 2025 12:55
Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would like to test this with a real extension first. But this lgtm

@mamhoff
Copy link
Contributor Author

mamhoff commented May 5, 2025

Closing in favor of #234

@mamhoff mamhoff closed this May 5, 2025
@mamhoff mamhoff deleted the rubocop-plugins branch May 5, 2025 13:43
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