Skip to content

Update cucumber #2518

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 2 commits into from
Sep 18, 2021
Merged

Update cucumber #2518

merged 2 commits into from
Sep 18, 2021

Conversation

pirj
Copy link
Member

@pirj pirj commented Sep 18, 2021

cucumber 4.0.0 is not aware of breaking changes in diff-lcs 1.4.3, so excluding it.

cucumber 3.2 would run on Ruby 2.2, 2.3 and 2.4.

Running cucumber was not excluding features that don't match the current Rails version (@rails_pre_6/@rails_post_6). rake cucumber was working fine.
I've reworked this from Rakefile to be a Before Cucumber hook in features/support.

Sibling PRs:

@pirj pirj self-assigned this Sep 18, 2021
@pirj pirj requested review from benoittgt and JonRowe September 18, 2021 14:39
cucumber 4.0.0 is not aware of breaking changes in diff-lcs 1.4.3, so
excluding it

cucumber 3.2 would run on Ruby 2.2, 2.3 and 2.4

Sibling PRs:
 - rspec/rspec-core#2877
 - rspec/rspec-mocks#1439
 - rspec/rspec-expectations#1320
Otherwise, running `cucumber` won't filter scenarios, only `rake
cucumber` will.
@JonRowe JonRowe merged commit 211d7d9 into main Sep 18, 2021
@JonRowe JonRowe deleted the update-cucumber branch September 18, 2021 20:50
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.

2 participants