-
Notifications
You must be signed in to change notification settings - Fork 529
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
Add support for Rails 7.2 #563
Conversation
v2.6.4 doesn't actually support Rails 7.2, so drop this item from the changelog.
Checks all passing. Thank you @andrewhampton. |
@@ -2,8 +2,6 @@ | |||
|
|||
## 2.6.4 - July 20, 2024 | |||
|
|||
Add support for [Rails 7.2](https://github.com/rails/rails/releases/tag/v7.2.0) |
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.
?
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.
the 3 PRs listed below was about supporting changes in 7.2, thus that title here
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.
Ah, 2.6.4 didn't actually support 7.2. That combined with seeing this comment made me think this line was added by mistake. So I removed it as part of this PR.
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.
make sense, I should have been more clear about it. I could not bump the version until it was officially released. but those were preliminary fix that was shown to be required for those using the pre-release.
This adds support for Rails 7.2.
Everything is pretty straightforward with the exception of the required version of
sqlite3
. I see theSQLITE_VERSION
env var can be used to specify a specific version of sqlite3, so maybe I should use that for testing rails 7.2?closes #562