Skip to content

Conversation

AlistairNorman
Copy link

Summary

The versions we were requiring of rspec-rails did not match up with the current versions of Rails that are under test for this gem so I updated rspec-rails versions.

Checklist

Check out our PR guidelines for more details.

The following are mandatory for all PRs:

The following are not always needed:

  • 📖 I have updated the README to account for my changes.
  • 📑 I have documented new code with YARD.
  • 🛣️ I have opened a PR to update the guides.
  • ✅ I have added automated tests to cover my changes.
  • 📸 I have attached screenshots to demo visual changes.

These versions were very old. Version 5.x is for Rails 5.2 and
6.x. 7.0 was removed from our CI tests so these are much older than we
are testing. We added Rails 8 to the CI tests which means we should be
using rspec-rails 8.x.
@AlistairNorman AlistairNorman force-pushed the alistair/update-rspec-rails branch 2 times, most recently from 8e25a6f to 6faee34 Compare September 3, 2025 17:16
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.

3 participants