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: Use Ruby 3.3 for linting #35

Merged
merged 7 commits into from
Jan 9, 2024
Merged

CI: Use Ruby 3.3 for linting #35

merged 7 commits into from
Jan 9, 2024

Conversation

olleolleolle
Copy link
Member

@olleolleolle olleolleolle commented Jan 9, 2024

RuboCop is set to use 2.6 syntax rules in any case.


Turned out to need a newer RuboCop, and newer plugins, and after that, it was ready to be used.

@iMacTia
Copy link
Member

iMacTia commented Jan 9, 2024

Next release will drop Ruby 2.6 and 2.7 anyway, so feel free to remove them in this PR if it helps getting CI green 👍

@olleolleolle
Copy link
Member Author

The real reason was "used a bundled Bundler which had some older issues", but now, we got rid of the 2.6, 2.7 from the CI matrix. Should we bump the gemspec min required Ruby version, too, @iMacTia ?

@iMacTia
Copy link
Member

iMacTia commented Jan 9, 2024

Yes we'll need to bump it in the gem spec and also in the rubocop.yml file.
Also, I wanted to check if we need to put a version restriction on net-http.
All these can go into a separate PR though, happy to get this merged in the meantime to unblock CI

@olleolleolle
Copy link
Member Author

@iMacTia I will refrain from checking for version restriction on the net-http gem, but the rest of the things are here, and I will merge.

@olleolleolle olleolleolle merged commit 69985d1 into main Jan 9, 2024
5 checks passed
@olleolleolle olleolleolle deleted the olleolleolle-patch-2 branch January 9, 2024 09:49
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