Skip to content
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

Breaking: Require ruby 2.3 or newer, to be able to use latest Cucumber, for 2.7 support #816

Merged
merged 9 commits into from
Apr 12, 2020

Conversation

olleolleolle
Copy link
Member

@olleolleolle olleolleolle commented Apr 12, 2020

This PR updates the required_ruby_version to >= 2.3, in order to be able to run Cucumber tests.

  • Add Ruby 2.7 to test matrix - Fixes CI: Add 2.7 to the matrix #787
  • Prepare for Ruby 2.7 compatibility
  • Also: Update the excon library - Fixes Allow Excon >= 0.62.0 #814
  • Update cucumber to latest release
  • Modernize Cucumber configuration
  • Breaking: Drop Cucumber 1.3 support
  • Breaking: Update the required_ruby_version to >= 2.3

@olleolleolle olleolleolle requested a review from a team April 12, 2020 14:00
@olleolleolle
Copy link
Member Author

Cc @mvz

@olleolleolle olleolleolle marked this pull request as ready for review April 12, 2020 14:06
@olleolleolle olleolleolle marked this pull request as draft April 12, 2020 14:48
@olleolleolle
Copy link
Member Author

olleolleolle commented Apr 12, 2020

Update: Learning that the "features/test_frameworks/cucumber_feature" does not currently find more than 0 tests. Investigating.

Learned: 2.4.0 won't work, so ditching that special-case "lower-version of Cucumber" test would make us go green (while still running tests).

I kept a "Gemfile-cucumber-3.1", which is the current gemspec's choice, to see if we are able to support 4.0 and 3.1 when 4.0 drops.

@olleolleolle olleolleolle marked this pull request as ready for review April 12, 2020 15:14
Copy link
Contributor

@mvz mvz left a comment

Choose a reason for hiding this comment

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

👍

spec/lib/vcr/cassette/serializers_spec.rb Outdated Show resolved Hide resolved
@krainboltgreene krainboltgreene merged commit b38369d into master Apr 12, 2020
@krainboltgreene krainboltgreene deleted the require-ruby-23-and-up branch April 12, 2020 21:41
@choosen
Copy link
Contributor

choosen commented May 29, 2020

Can you mention explicit about support ruby 2.7?
I like that there is info about 2.3 >= but explicit 2.7 would be great

@olleolleolle
Copy link
Member Author

@choosen I'd merge a small PR if you have one?

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

4 participants