-
Notifications
You must be signed in to change notification settings - Fork 5
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
Drop support for Rails < 6, add support for Rails 8 & Ruby 3.4 #15
Conversation
ecc0505
to
efcc9fa
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
@@ -1,15 +1,15 @@ | |||
source 'https://rubygems.org' | |||
|
|||
# Runtime dependencies | |||
gem 'activerecord', '~>5' | |||
gem 'mysql2', '~>0.4.10' |
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.
Wieso schmeißen wir eigentlich MySQL für PG raus?
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 removed (among others) the Gemfile.5.2.mysql2
and and created the new Gemfile.8.0.pg
. The changes apparently where so few, that git thought I moved the file. That's why we have mysql changes in the pg file.
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.
When removing stuff, you'll never find overlooked stuff in a merge request 😬 But looks fine. When tests are green, should be alright.
CHANGELOG.md
Outdated
@@ -7,10 +7,12 @@ This project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html | |||
## Unreleased | |||
|
|||
### Breaking changes | |||
- Drop support for Rails < 6.1 |
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'd append "This also drops support for MySQL, which had only been present for Rails < 6."
233a4ab
to
5310858
Compare
No description provided.