-
Notifications
You must be signed in to change notification settings - Fork 35
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
Upgrade rails to 7.2.x version; clean up unsupported rails and ruby version #64
Conversation
ActiveRecord::MigrationContext.new(migrations_path, ActiveRecord::SchemaMigration).migrate | ||
end | ||
|
||
if active_record_version >= Gem::Version.new("7.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.
Rails has been back and forth about how to use the MigrationContext https://stackoverflow.com/questions/60974746/what-do-i-pass-to-schema-migration-in-activerecordmigrationcontextnew
8ffff44
to
29af2fe
Compare
@@ -5,3 +5,4 @@ source "https://rubygems.org" | |||
gemspec path: ".." | |||
|
|||
gem "activerecord", github: "rails/rails", branch: "6-1-stable" | |||
gem "sqlite3", "~> 1.4" |
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 newer version (2.x.x) is released but the older rails versions are still pin to use a lower version (~> 1.4).
Reference PR
Only the latest version (7.2.x) is able to use a higher version.
@@ -5,3 +5,4 @@ source "https://rubygems.org" | |||
gemspec path: ".." | |||
|
|||
gem "activerecord", github: "rails/rails", branch: "7-0-stable" | |||
gem "sqlite3", "~> 1.4" |
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.
Same as above.
29af2fe
to
6253f67
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.
Not seeing any issuse and the config change I suggested has been made.
To resolve the requested issue.
Also, clean up unsupported rails version (< 6.1) and ruby version(< 3.0)