Skip to content

[CIVIS-14025] Add support for Rails 8.1 - #8

Merged
gskifstad merged 1 commit into
mainfrom
CIVIS-14025-rails-8.1
Aug 26, 2026
Merged

gskifstad merged 1 commit into
mainfrom
CIVIS-14025-rails-8.1

Conversation

@gskifstad

Copy link
Copy Markdown

Summary

  • Adds gemfiles/8.1.gemfile and a 8.1 entry in the CI matrix so the gem is tested against Rails 8.1.
  • Bumps the gem version to 5.1.0 and adds a corresponding CHANGELOG.markdown entry.
  • Documents, in the README, which files to update when adding support for a future Rails version.

Why no code changes

Authority's Rails touchpoints (ActiveSupport::Concern, ActiveSupport::Rescuable, Rails.root, ActionController::Base) are unaffected by Rails 8.1, and the gemspec's existing activesupport '< 9' bound already permits it. I confirmed this by building a Docker image (Ruby 3.4 + Rails 8.1.3.1) via the repo's existing docker-compose/start_container.sh setup and running the full spec suite against it: 160 examples, 0 failures, with no changes to lib/.

Test plan

  • bundle exec rspec passes against Rails 8.1.3.1 in Docker (Ruby 3.4)
  • CI matrix updated to cover 8.1 going forward

Authorship

This PR was written by Claude Code (AI) at my direction, based on my review of the diff. All changes are mechanical (gemfile, CI matrix, version bump, changelog, README docs) — I verified the passing test run and reviewed the README additions before requesting this PR.

🤖 Generated with Claude Code

Verified via Docker (Ruby 3.4 + Rails 8.1.3.1) that the full spec suite
passes unmodified, since Authority's Rails touchpoints are unaffected
by 8.1. Wires 8.1 into the gemfile matrix and CI, bumps the gem
version, and documents which files to update for future Rails version
bumps.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@gskifstad gskifstad added the > 90% ai generated used to tag prs written primarily by a coding agent label Aug 25, 2026
@gskifstad gskifstad changed the title Add support for Rails 8.1 [CIVIS-14025] Add support for Rails 8.1 Aug 25, 2026
@gskifstad
gskifstad requested review from a team and leanne73 and removed request for a team August 25, 2026 17:12

@leanne73 leanne73 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.

LGTM

@gskifstad
gskifstad merged commit f789765 into main Aug 26, 2026
25 checks passed
@gskifstad
gskifstad deleted the CIVIS-14025-rails-8.1 branch August 26, 2026 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

> 90% ai generated used to tag prs written primarily by a coding agent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants