Skip to content

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
@mergify
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.
@mamhoff mamhoff force-pushed the allow-rspec-rails-6 branch from 394651d to ba2b820 Compare January 18, 2024 13:21
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
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.

2 participants