Skip to content

Prep version 5 #2476

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

Merged
merged 4 commits into from
Mar 9, 2021
Merged

Prep version 5 #2476

merged 4 commits into from
Mar 9, 2021

Conversation

JonRowe
Copy link
Member

@JonRowe JonRowe commented Mar 6, 2021

As we prepare to release Rails 6.1 support, following our new versioning strategy this means changing the major version and matching the supported Rails versions, see https://guides.rubyonrails.org/maintenance_policy.html, which is 5.2+

So this is for review of that new constraint for us, and removes a bunch of 4.2 / 5 supporting conditionals.

@JonRowe JonRowe requested review from pirj and benoittgt March 6, 2021 10:29
@pirj
Copy link
Member

pirj commented Mar 6, 2021 via email

[Full Changelog](https://github.com/rspec/rspec-rails/compare/v4.1.0...main)

Enhancements:

* Support new #file_fixture_path and new fixture test support code. (Jon Rowe, #2398)
* Support for Rails 6.1. (Benoit Tigeot, Jon Rowe, Phil Pirozhkov, and more #2398)
Copy link
Member

Choose a reason for hiding this comment

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

Should we add more details about this?

Copy link
Member Author

Choose a reason for hiding this comment

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

Please do!

Copy link
Member Author

Choose a reason for hiding this comment

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

@benoittgt and @pirj, I intend to merge this and release 5.0 tomorrow unless either of you have anything you want to add (Im happy to delay a bit longer if so). :)

Copy link
Member

Choose a reason for hiding this comment

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

After my message I looked at the commit list and found commits pretty technical or deprecation fix. I am wondering how much it is valuable to be in the changelog. 🤔

I have no blocker for a release. I can work for the a blog post on rspec.info next week with infos about this release.

Thanks for the work.

Copy link
Member

@pirj pirj left a comment

Choose a reason for hiding this comment

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

Everything seems to be in order, thank you!

This was referenced Mar 15, 2021
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.

None yet

3 participants