Skip to content

Backport Devise 5 support#8964

Merged
tagliala merged 1 commit into3-0-stablefrom
feature/3-0-stable-devise5-support
Mar 8, 2026
Merged

Backport Devise 5 support#8964
tagliala merged 1 commit into3-0-stablefrom
feature/3-0-stable-devise5-support

Conversation

@tagliala
Copy link
Contributor

@tagliala tagliala commented Mar 7, 2026

Ref: #8936

@codecov
Copy link

codecov bot commented Mar 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (3-0-stable@e79279a). Learn more about missing BASE report.

Additional details and impacted files
@@              Coverage Diff              @@
##             3-0-stable    #8964   +/-   ##
=============================================
  Coverage              ?   99.19%           
=============================================
  Files                 ?      194           
  Lines                 ?     4971           
  Branches              ?        0           
=============================================
  Hits                  ?     4931           
  Misses                ?       40           
  Partials              ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tagliala tagliala requested a review from mgrunberg March 7, 2026 23:00
@tagliala
Copy link
Contributor Author

tagliala commented Mar 7, 2026

I've also tested this locally.

Seems fine

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR backports Devise 5 compatibility by widening ActiveAdmin’s runtime Devise version guard and updating the repository’s appraisal/development gemfiles and lockfiles to exercise Devise 5 + Formtastic 6 on Rails 8.x while keeping older Rails appraisals on Devise 4.9.

Changes:

  • Allow Devise 5 at runtime by widening the ActiveAdmin::Dependency::Requirements::DEVISE constraint to < 6.
  • Update Rails 8.x Gemfiles/lockfiles (and the root Gemfile/lockfile) to use Devise ~> 5.0 and Formtastic >= 6.0.0.
  • Keep Rails 6.1–7.2 appraisals pinned to Devise ~> 4.9 (and, for Rails 7.2, Formtastic ~> 5.0) due to Ruby 3.1 coverage.

Reviewed changes

Copilot reviewed 7 out of 13 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
lib/active_admin/dependency.rb Widens allowed Devise versions to include 5.x.
Gemfile Pins development/test dependencies to Devise 5 and Formtastic 6 for Rails 8.1.
Gemfile.lock Updates lockfile to Devise 5/Formtastic 6 (plus other resolved upgrades).
gemfiles/rails_80/Gemfile Pins Rails 8.0 appraisal to Devise 5 and Formtastic 6.
gemfiles/rails_80/Gemfile.lock Locks Rails 8.0 appraisal with Devise 5/Formtastic 6 (plus other resolved upgrades).
gemfiles/rails_72/Gemfile Keeps Devise 4.9 and pins Formtastic ~>5.0 for Ruby 3.1 compatibility.
gemfiles/rails_72/Gemfile.lock Updates Rails 7.2 appraisal lockfile and direct dependency constraints.
gemfiles/rails_71/Gemfile Keeps Devise 4.9 for Rails 7.1 appraisal.
gemfiles/rails_71/Gemfile.lock Updates Rails 7.1 appraisal lockfile (plus other resolved upgrades).
gemfiles/rails_70/Gemfile Keeps Devise 4.9 for Rails 7.0 appraisal.
gemfiles/rails_70/Gemfile.lock Updates Rails 7.0 appraisal lockfile (plus other resolved upgrades).
gemfiles/rails_61/Gemfile Pins Rails 6.1 appraisal to Devise ~>4.9.
gemfiles/rails_61/Gemfile.lock Updates Rails 6.1 appraisal lockfile and pins Devise ~>4.9 in dependencies.

@tagliala tagliala merged commit 34197f0 into 3-0-stable Mar 8, 2026
33 checks passed
@tagliala tagliala deleted the feature/3-0-stable-devise5-support branch March 8, 2026 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants