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

Feature: Cucumber 3.x Support #38

Merged
merged 4 commits into from
Apr 7, 2020

Conversation

todd-a-jacobs
Copy link

This is a minimalist pull-request that:

  1. bumps the minimum Cucumber version to 3.1
  2. replicates the missing dump_count method from the cucumber-ruby source code
  3. passes all existing specs

This appears to close #36 and stops the NoMethodError exception raised from Guard::Cucumber::NotificationFormatter.

- The dump_count method was previously found in
  Cucumber::Formatter::Summary and Cucumber::Formatter::Html
- This method was extracted from
  <cucumber/cucumber-ruby@4154c8ca>
- Closes guard#36
  <guard#36>
- Prevent dependencies on outdated gems.
- Cucumber 3.1 is now a minimum version, not a fixed dependency.
@todd-a-jacobs todd-a-jacobs changed the title Feature: Cucumber 3.x support Feature: Cucumber 3.x Support Dec 30, 2018
@anicholson
Copy link

This seems like the best minimal upgrade path to unblock folk without investigating what's needed to use #dump_count's replacement :)

@jeffshantz
Copy link

Thanks for this, @todd-a-jacobs.

@piotr-kedziak
Copy link

Hi, are this gem still maintained ?

@marnen
Copy link

marnen commented Jul 11, 2019

Will this be merged at some point?

@marnen
Copy link

marnen commented Nov 11, 2019

Can I help get this merged?

@marnen
Copy link

marnen commented Apr 2, 2020

This still should be merged. Again, I'll ask: what's the delay? Can I help somehow?

@rymai rymai self-assigned this Apr 7, 2020
@rymai
Copy link
Member

rymai commented Apr 7, 2020

@todd-a-jacobs Thank you, looks good to me! It looks like @e2 isn't maintaining the gem anymore so I'll merge and release a new version. ☀️ 🚀

@marnen
Copy link

marnen commented Apr 7, 2020

@rymai Thank you so much!

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.

Cucumber 3.0 compatibility
6 participants