rspec / rspec-core

RSpec runner and formatters
http://rspec.info
MIT License
1.23k stars 765 forks source link

Add deprecation warnings for features to be removed/changed in RSpec 4 #2880

Closed pirj closed 4 days ago

pirj commented 3 years ago

A few things are a bit off, e.g.:

This is purposed to be released as version 3.99. Sibling PRs:

Release strategy: https://github.com/rspec/rspec-core/pull/2880#issuecomment-797756681 RSpec 4 plan: https://github.com/rspec/rspec/issues/61 Changes: https://github.com/rspec/rspec-core/blob/4-0-dev/Changelog.md#development

TODO

pirj commented 3 years ago

@JonRowe Am I doing the right thing?

JonRowe commented 3 years ago

@pirj yep, thats the exactly the kind of thing we'll want in a final 3.x release.

We'll follow the 2.x strategy of:

It's sufficient to keep these as a PR until we get to releasing 3.99, the first two steps are just release management as usual

pirj commented 3 years ago

Green!

I have seen features/command_line/init.feature:25, and features/command_line/bisect.feature (on JRuby) failing intermittently on CI. Might be flaky. Couldn't reproduce locally.

benoittgt commented 3 years ago

Thanks @pirj for this PR. 🙌🏻

should we add deprecation warnings when running on rbx/ree/Ruby < 2.3?

It means people will always have the deprecations when they use the last version chosen by bundler for their Ruby version?

benoittgt commented 3 years ago

@pirj are we good to merge it?

Sorry I scrolled back and saw

It's sufficient to keep these as a PR until we get to releasing 3.99, the first two steps are just release management as usual

JonRowe commented 4 days ago

Migrated to the monorepo