Skip to content

Upgrade rails to 7.2.x version; clean up unsupported rails and ruby version - #64

Merged
wendy-clio merged 1 commit into
masterfrom
upgrade_rails_to_7_2
Jan 13, 2025
Merged

Upgrade rails to 7.2.x version; clean up unsupported rails and ruby version#64
wendy-clio merged 1 commit into
masterfrom
upgrade_rails_to_7_2

Conversation

@wendy-clio

@wendy-clio wendy-clio commented Nov 1, 2024

Copy link
Copy Markdown
Contributor

To resolve the requested issue.

Also, clean up unsupported rails version (< 6.1) and ruby version(< 3.0)

@wendy-clio
wendy-clio requested a review from a team November 1, 2024 14:04
@wendy-clio wendy-clio self-assigned this Nov 1, 2024
@wendy-clio
wendy-clio requested a review from a team as a code owner November 1, 2024 14:04
Comment thread spec/spec_helper.rb
ActiveRecord::MigrationContext.new(migrations_path, ActiveRecord::SchemaMigration).migrate
end

if active_record_version >= Gem::Version.new("7.0")

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wendy-clio
wendy-clio force-pushed the upgrade_rails_to_7_2 branch 3 times, most recently from 8ffff44 to 29af2fe Compare November 1, 2024 14:22
gemspec path: ".."

gem "activerecord", github: "rails/rails", branch: "6-1-stable"
gem "sqlite3", "~> 1.4"

Copy link
Copy Markdown
Contributor Author

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.

gemspec path: ".."

gem "activerecord", github: "rails/rails", branch: "7-0-stable"
gem "sqlite3", "~> 1.4"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as above.

Comment thread .github/workflows/ci.yml Outdated
@wendy-clio
wendy-clio force-pushed the upgrade_rails_to_7_2 branch from 29af2fe to 6253f67 Compare November 4, 2024 15:09
@wendy-clio
wendy-clio requested a review from RafeSl November 4, 2024 15:10

@RafeSl RafeSl left a comment

Copy link
Copy Markdown

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.

@wendy-clio
wendy-clio merged commit f86162e into master Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants