Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow rspec-rails 6 #217

Merged
merged 1 commit into from Jan 18, 2024
Merged

Allow rspec-rails 6 #217

merged 1 commit into from Jan 18, 2024

Conversation

mamhoff
Copy link
Contributor

@mamhoff mamhoff commented Jan 18, 2024

Summary

rspec-rails < 6.0.2 is incompatible with Rails 7.1 because of stricter view path typecasting in Rails 7.1. This bug is fixed in rspec-rails 6.0.2. Without this, controller specs in extensions will break.

rspec/rspec-rails#2631

Checklist

The following are mandatory for all PRs:

@mergify mergify bot added the needs changelog label Needs a label to determine the type of change. label Jan 18, 2024
Copy link
Contributor

mergify bot commented Jan 18, 2024

It looks like this PR is missing a label to determine the type of change it introduces. The maintainer should add one of the following labels:

  • bug for bugfixes.
  • enhancement for new features and improvements.
  • documentation for documentation changes.
  • security for security patches.
  • removed for feature removals.
  • infrastructure for internal changes that should not go in the changelog.

Additionally, the maintainer may also want to add one of the following:

  • breaking for breaking changes.
  • deprecated for feature deprecations.

Once the correct labels have been set, simply remove the needs changelog label label from this PR so I can merge it.

@tvdeyen tvdeyen added bug Describes or fixes a bug. and removed needs changelog label Needs a label to determine the type of change. labels Jan 18, 2024
rspec-rails < 6.0.2 is incompatible with Rails 7.1 because of stricter
view path typecasting in Rails 7.1. This bug is fixed in rspec-rails
6.0.2. Without this, controller specs in extensions will break.

rspec/rspec-rails#2631

This also raises the minimum version to rspec-rails 5 as Solidus 3.x
needs Rails > 5.2, and rspec-rails 5 is the version for that Rails
version.
Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

Thanks!

@mergify mergify bot merged commit 197265b into solidusio:main Jan 18, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Describes or fixes a bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants