Skip to content

SW-6725: Allow Rails 7.x (raise gemspec rails ceiling to < 8.0)#5

Open
slad-cloudapp wants to merge 1 commit into
mainfrom
rails-7-2
Open

SW-6725: Allow Rails 7.x (raise gemspec rails ceiling to < 8.0)#5
slad-cloudapp wants to merge 1 commit into
mainfrom
rails-7-2

Conversation

@slad-cloudapp

Copy link
Copy Markdown

Why?

dropper_api is upgrading Rails 6.1 → 7.2.3.1 (SW-6714). This fork's gemspec caps the Rails dependency at <= 7.1, so bundler cannot resolve scim_rails against Rails 7.2 — a hard blocker for the upgrade.

What?

One-line change to scim_rails.gemspec:

- s.add_dependency "rails", ">= 6.0", "<= 7.1"
+ s.add_dependency "rails", ">= 6.0", "< 8.0"

Raises the ceiling to allow all of Rails 7.x. No functional/code changes to the gem — the SCIM controllers, config, and group functionality are untouched.

Caveats

  • Widens the supported range to Rails 7.x; not yet validated against Rails 8 (kept < 8.0 intentionally).
  • dropper_api currently pins to this branch (gem "scim_rails", github: "cloudapp/scim_rails", branch: "rails-7-2"). Once this merges to main, that PR will drop the branch: pin. Please do not delete the rails-7-2 branch until dropper_api's SW-6714 has merged and re-locked — the production bundle resolves against it in the meantime.

Testing Notes

  • SCIM provisioning/deprovisioning is exercised via dropper_api's SCIM specs on the Rails 7.2 branch (SW-6714), which resolves against this branch and boots/tests cleanly.

Tracked by SW-6725.

Copilot AI review requested due to automatic review settings July 15, 2026 02:46

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 scim_rails gem dependency constraints so it can be resolved alongside Rails 7.2.x (and generally all Rails 7.x) in downstream apps, unblocking the Rails 6.1 → 7.2 upgrade noted in the PR description.

Changes:

  • Relax the Rails dependency upper bound from <= 7.1 to < 8.0 in the gemspec.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread scim_rails.gemspec
Comment on lines 19 to +20
s.required_ruby_version = ">= 2.4"
s.add_dependency "rails", ">= 6.0", "<= 7.1"
s.add_dependency "rails", ">= 6.0", "< 8.0"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants