Skip to content
This repository was archived by the owner on Nov 30, 2024. It is now read-only.

Conversation

pirj
Copy link
Member

@pirj pirj commented Sep 5, 2021

1.3-2.0 don't work for me locally with a weird error:

undefined method `[]' for nil:NilClass (NoMethodError) /Users/pirj/.rvm/gems/ruby-2.7.1@rspec-core/gems/cucumber-1.3.20/lib/cucumber/core_ext/proc.rb:17:in `file_colon_line'

2.4.0 spits "undefined method with_filtered_backtrace" in some other repository (rspec-core to my best memory)
3.2.0 - "undefined method ok?" there as well
4.1 depends on diff-lcs (< 1.4, >= 1.3, ~> 1.3), while we depend on = 1.4.4

5.2.0 only supports Ruby > 2.5. we support >=2.3. But it depends on diff-lcs >= 1.4.4.

I've checked 3.2.0, 4.1.0, 5.3.0, 6.1.0, and they work (except for 4.1.0).

I vaguely recall we were discussing not running cucumber on older Rubies (2.3-2.4), is this correct?
Otherwise I'll roll back to 3.2.0.

Sibling PRs:

@pirj
Copy link
Member Author

pirj commented Sep 11, 2021

Removed everything unrelated to the version update. Green.

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

cucumber 4.0.0 would run on Ruby 2.3, 2.4 and JRuby 9.1
@pirj pirj merged commit 8372ed2 into 4-0-dev Sep 13, 2021
@JonRowe JonRowe deleted the update-cucumber branch September 14, 2021 06:56
pirj added a commit to rspec/rspec-rails that referenced this pull request 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

Sibling PRs:
 - rspec/rspec-core#2877
 - rspec/rspec-mocks#1439
 - rspec/rspec-expectations#1320
pirj added a commit to rspec/rspec-rails that referenced this pull request 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

Sibling PRs:
 - rspec/rspec-core#2877
 - rspec/rspec-mocks#1439
 - rspec/rspec-expectations#1320
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants