Skip to content

build: bump tested Rails versions to 7.1.6, 7.2.3.2, 8.0.5.1, 8.1.3.1 - #175

Merged
eitoball merged 2 commits into
masterfrom
bump-rails-test-versions
Sep 5, 2026
Merged

build: bump tested Rails versions to 7.1.6, 7.2.3.2, 8.0.5.1, 8.1.3.1#175
eitoball merged 2 commits into
masterfrom
bump-rails-test-versions

Conversation

@eitoball

Copy link
Copy Markdown
Contributor

What

Update the CI/appraisal Rails test matrix from 6.0.6/6.1.7.6/7.0.8/7.1.3 to 7.1.6/7.2.3.2/8.0.5.1/8.1.3.1.

Why

The previously tested Rails lines (6.0, 6.1, 7.0) are EOL, and Rails 8.0/8.1 were not covered at all. This shifts the tested window to the latest patch releases of 7.1, adds 7.2, and adds both current Rails 8 minor lines.

Changes

  • Appraisals: replace the four rails*/active* appraise blocks for 6.0.6/6.1.7.6/7.0.8/7.1.3 with new ones for 7.1.6/7.2.3.2/8.0.5.1/8.1.3.1
  • gemfiles/: regenerate the corresponding .gemfile/.gemfile.lock pairs via bundle exec appraisal generate + bundle lock (old version files removed, matched ruby/x86_64-linux platform convention used by the existing lockfiles)
  • .github/workflows/build.yml: update the gemfile matrix to the new appraisal names
  • README.md: update the stated tested Rails range (6.0–7.1 → 7.1–8.1) and the example appraisal command's Rails version

Notes

Rails 8.x requires Ruby >= 3.2, so its lockfiles were resolved under Ruby 3.4 (matches the Ruby versions already in the CI matrix). Local Xcode/CLT on this machine is broken, so the full bundle exec appraisal rspec run per gemfile could not be executed locally (dependency resolution succeeded and the default non-Rails spec suite passes); CI will validate the native-extension build and full spec run across the matrix.

Co-Authored-By: Claude Sonnet 5 noreply@anthropic.com

Drop the EOL 6.0, 6.1, and 7.0 lines and add Rails 8.0/8.1 coverage,
regenerating the Appraisal gemfiles and locks accordingly.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 30, 2026 09:02

Copilot AI 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.

Pull request overview

Updates the gem’s Appraisal/CI Rails + ActiveRecord test matrix to drop EOL Rails lines and add coverage for Rails 7.2 and Rails 8.0/8.1, with regenerated appraisal gemfiles/lockfiles to match.

Changes:

  • Bumped Appraisal definitions and regenerated gemfiles/ for Rails/ActiveRecord 7.1.6, 7.2.3.2, 8.0.5.1, and 8.1.3.1 (removing the old 6.0/6.1/7.0 gemfiles).
  • Updated GitHub Actions workflow matrix to run against the new appraisal gemfile names.
  • Updated README to reflect the new tested Rails range and example appraisal command.

Reviewed changes

Copilot reviewed 11 out of 23 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
README.md Updates documented tested Rails range and appraisal example.
Appraisals Replaces old Rails/AR appraisal blocks with 7.1.6/7.2.3.2/8.0.5.1/8.1.3.1.
.github/workflows/build.yml Updates CI gemfile matrix entries to the new appraisal names.
gemfiles/rails7.1.6.gemfile Pins Rails 7.1.6 for the Rails integration appraisal.
gemfiles/rails7.1.6.gemfile.lock Lockfile for Rails 7.1.6 appraisal.
gemfiles/rails7.2.3.2.gemfile Pins Rails 7.2.3.2 for the Rails integration appraisal.
gemfiles/rails7.2.3.2.gemfile.lock Lockfile for Rails 7.2.3.2 appraisal.
gemfiles/rails8.0.5.1.gemfile Pins Rails 8.0.5.1 for the Rails integration appraisal.
gemfiles/rails8.0.5.1.gemfile.lock Lockfile for Rails 8.0.5.1 appraisal.
gemfiles/rails8.1.3.1.gemfile Pins Rails 8.1.3.1 for the Rails integration appraisal.
gemfiles/rails8.1.3.1.gemfile.lock Lockfile for Rails 8.1.3.1 appraisal.
gemfiles/active7.1.6.gemfile Pins ActiveRecord/ActiveSupport 7.1.6 for the AR-only appraisal.
gemfiles/active7.1.6.gemfile.lock Lockfile for AR-only 7.1.6 appraisal.
gemfiles/active7.2.3.2.gemfile Pins ActiveRecord/ActiveSupport 7.2.3.2 for the AR-only appraisal.
gemfiles/active7.2.3.2.gemfile.lock Lockfile for AR-only 7.2.3.2 appraisal.
gemfiles/active8.0.5.1.gemfile Pins ActiveRecord/ActiveSupport 8.0.5.1 for the AR-only appraisal.
gemfiles/active8.0.5.1.gemfile.lock Lockfile for AR-only 8.0.5.1 appraisal.
gemfiles/active8.1.3.1.gemfile Pins ActiveRecord/ActiveSupport 8.1.3.1 for the AR-only appraisal.
gemfiles/active8.1.3.1.gemfile.lock Lockfile for AR-only 8.1.3.1 appraisal.
gemfiles/rails7.0.8.gemfile.lock Removes old Rails 7.0.8 lockfile.
gemfiles/rails6.1.7.6.gemfile.lock Removes old Rails 6.1.7.6 lockfile.
gemfiles/rails6.0.6.gemfile.lock Removes old Rails 6.0.6 lockfile.
gemfiles/active6.0.6.gemfile.lock Removes old AR-only 6.0.6 lockfile.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/build.yml
Comment thread README.md Outdated
Copilot AI review requested due to automatic review settings August 30, 2026 09:07

Copilot AI 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.

Pull request overview

Copilot reviewed 11 out of 23 changed files in this pull request and generated no new comments.

Suppressed comments (1)

README.md:13

  • This sentence implies Rails 8.0/8.1 are tested on Ruby 3.1, but the CI matrix excludes Ruby 3.1 for the Rails 8 appraisals (and Rails 8.x requires Ruby >= 3.2). Consider clarifying the Ruby requirement for the Rails 8 lines to avoid misleading readers.
For Rails / ActiveRecord integration, this repository is currently tested against ActiveRecord and Rails 7.1 through 8.1 on Ruby 3.1 through 4.0.

…y 3.2

Rails 8.0/8.1 require Ruby >= 3.2. Also re-resolve the Rails 8 lockfiles
under Ruby 3.2 instead of 3.4 so they don't pin rdoc/rbs versions that
require a newer Ruby than the matrix's minimum supported version.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@eitoball
eitoball force-pushed the bump-rails-test-versions branch from 32445e5 to 994b77b Compare September 5, 2026 08:18
Copilot AI review requested due to automatic review settings September 5, 2026 08:18

Copilot AI 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.

🔵 Needs a closer look

Large dependency and CI-matrix shifts across multiple Rails major/minor lines should be confirmed by CI signal before approval.

Review details
  • Files reviewed: 11/23 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

@eitoball
eitoball merged commit c1f961e into master Sep 5, 2026
73 checks passed
@eitoball
eitoball deleted the bump-rails-test-versions branch September 5, 2026 08:41
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