-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Remove Ruby v2.5 support #4845
Remove Ruby v2.5 support #4845
Conversation
71ff2f3
to
e3090f7
Compare
Ruby v2.5 is EOL since 05 Apr 2021 [1]. Our new release policy [2] makes it clear that we don't support Ruby versions that are no longer maintained. [1] - https://www.ruby-lang.org/en/downloads/branches/ [2] - https://solidus.io/release_policy/
The new syntax was introduced back on Ruby v2.6 [1] [1] - https://ruby-doc.org/core-2.6/Range.html#class-Range-label-Endless+Ranges
`Object#yield_self` is available from Ruby 2.6 [1] [1] - https://ruby-doc.org/core-2.6/Object.html#method-i-yield_self
e3090f7
to
1f11c28
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.
👍
@@ -18,7 +18,7 @@ def build_gem_lines(conservative_versioning) | |||
requirement = if is_local | |||
", path: \"#{d.source.path}\"" | |||
elsif is_git | |||
", git: \"#{d.git}\"".yield_self { |g| d.ref ? g + ", ref: \"#{d.ref}\"" : g } | |||
", git: \"#{d.git}\"".then { |g| d.ref ? g + ", ref: \"#{d.ref}\"" : g } |
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.
😍
|
I did that just now to be able to merge this PR. |
Summary
Ruby v2.5 is EOL since 05 Apr 2021.
Our new release policy makes it clear that we don't support Ruby versions that are no longer maintained.
We also make a basic sanitization check in our code base to update to newer standards:
Closes #4840
Checklist
Check out our PR guidelines for more details.
The following are mandatory for all PRs:
The following are not always needed (
cross them outif they are not):[ ] I have added automated tests to cover my changes.[ ] I have attached screenshots to demo visual changes.[ ] I have opened a PR to update the guides.[ ] I have updated the README to account for my changes.