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

ci: Add Rubocop checks to CI and use latest versions #1119

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

nschonni
Copy link
Contributor

Noticed that it wasn't currently called in the CI, so I updated to the latest version, and then added a job in the CI to make sure it's called during the PR process

@BobbyMcWho
Copy link
Member

I honestly don't strictly enforce rubocop on this repo any more, it's not under super active development, so it's a bit more hassle than it's worth

@nschonni
Copy link
Contributor Author

@BobbyMcWho my thinking was for the automatic rewrites it can do if the minimum Ruby version get's bumped to v3 when updating TargetRubyVersion

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.

None yet

2 participants