-
Notifications
You must be signed in to change notification settings - Fork 28
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
Support RSpec 4 #64
Support RSpec 4 #64
Conversation
RSpec 4 has changed the semantic of multi-condition filtering. To include a module to a group matching *any* of the conditions, the conditions should be separate statements.
@alexrothenberg Appreciate it if you could accept and merge. RSpec 4's development build is red. |
See: - rspec/rspec-core#1821 - rspec/rspec-core#2874 - alexrothenberg/ammeter#64 - example failure (Ammeter::RSpec::Rails::GeneratorExampleGroup that defines destination is not included) https://github.com/rspec/rspec-core/pull/2874/checks?check_run_id=1942170588 Alternative to #2468
Spec failures look unrelated:
|
See: - rspec/rspec-core#1821 - rspec/rspec-core#2874 - alexrothenberg/ammeter#64 - example failure (Ammeter::RSpec::Rails::GeneratorExampleGroup that defines destination is not included) https://github.com/rspec/rspec-core/pull/2874/checks?check_run_id=1942170588 Alternative to #2468
See: - rspec/rspec-core#1821 - rspec/rspec-core#2874 - alexrothenberg/ammeter#64 - example failure (Ammeter::RSpec::Rails::GeneratorExampleGroup that defines destination is not included) https://github.com/rspec/rspec-core/pull/2874/checks?check_run_id=1942170588 Alternative to #2468
See: - rspec/rspec-core#1821 - rspec/rspec-core#2874 - alexrothenberg/ammeter#64 - example failure (Ammeter::RSpec::Rails::GeneratorExampleGroup that defines destination is not included) https://github.com/rspec/rspec-core/pull/2874/checks?check_run_id=1942170588 Alternative to #2468
@pirj Thank you for this. I'll merge and try to get a new version onto rubygems as soon as I can get the build green again. As you've figured out I haven't touched this in a long time and bitrot has caused CI to go red but I should be able to resolve that with a little time. |
Thank you! We've worked this around with mininal impact in If you need help with any of the build issues - let me know, chances are we've been though some of this in RSpec, and I could help fixing or point out at our fixes. |
Thanks for the offer. I used the github actions https://github.com/rspec/rspec-rails/blob/main/.github/workflows/ci.yml from rspec-rails as inspiration and was able to get it working in #65. I've already merged and pushed v1.1.5 up to https://rubygems.org/gems/ammeter. I'm so happy to know that the rspec team still finds this useful as I use rspec every day and appreciate all the hard work you all have done to keep it awesome all these years 😍 |
Yes, Thank you once again, for your support and updating it so quickly! |
RSpec 4 has changed the semantic of multi-condition filtering. To include a module to a group matching any of the conditions, the conditions should be separate statements.
See:
Ammeter::RSpec::Rails::GeneratorExampleGroup
that definesdestination
is not included) https://github.com/rspec/rspec-core/pull/2874/checks?check_run_id=1942170588