-
Notifications
You must be signed in to change notification settings - Fork 116
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
Bump the range of supported Ruby and Rails versions #182
Bump the range of supported Ruby and Rails versions #182
Conversation
1aadc76
to
58686f2
Compare
Forgot to mention it in the changelog. Added it 🙂 |
@sato11 could you please fix the conflicts? 🙏 Thanks! |
58686f2
to
dadbf9a
Compare
.github/workflows/ci.yml
Outdated
@@ -36,8 +36,6 @@ jobs: | |||
exclude: | |||
- ruby: 3.0 | |||
gemfile: gemfiles/rails_5_2.gemfile |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can also drop Rails 5.0 and 5.2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right. Will commit it here since you mention it 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nbulaj How does it look to you now? 🙂
Ruby 3.2 is out and 2.6 has reached the EOL: https://www.ruby-lang.org/en/downloads/branches/ Also since Rails 5 has been dropped support for and corresponding gemfiles are now dispensable to ci.
dadbf9a
to
b810c44
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Ruby 3.2 is out and 2.6 has reached the EOL:
https://www.ruby-lang.org/en/downloads/branches/
Also since Rails 5 has been dropped support for and
corresponding gemfiles are now dispensable to ci.
This is a follow-up to what I have committed to doorkeeper gem in doorkeeper-gem/doorkeeper#1618. I guess it would be great if the CI setup mirrors what doorkeeper has just like #175 has achieved it.
As 2.6 is dropped from CI I'm including the change that bumps the required ruby version. What do you think?