-
-
Notifications
You must be signed in to change notification settings - Fork 50
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
gherkin/parser
seems to be missing
#160
Comments
It sounds like you have somehow ended up with gherkin 6.0.10 installed, which is backwards incompatible. cucumber-ruby does specify |
This is where foodcritic pulls it in: https://github.com/Foodcritic/foodcritic/blob/v14.1.0/foodcritic.gemspec#L22 which is not safe |
We can probably trace it, I don't have the time right now but maybe a little later this evening I will: https://rubygems.org/gems/gherkin/reverse_dependencies |
According to https://rubygems.org/gems/cucumber-core/versions/3.2.0 its pulling in |
So the released gem specifies it as ☝️ https://github.com/cucumber/cucumber-ruby-core/blob/v3.2.0/cucumber-core.gemspec#L17 but the master has it pessemistically pinned. I can put together a PR to pin it, wait to have it released, then we should be able to release a v4 that pulls in the new dependency as |
That sounds great @majormoses! Thanks for digging into this. If you find other dangerously permissive dependencies, feel free to fix those too. |
Upgrading to v6 is underway btw: cucumber/cucumber-ruby#1313 |
Given a few people have experienced this today, and there's likely to be more over the coming days, I've released v3.2.1 (with just the pessimistic version change). :) |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Summary
It looks like the
gherkin
parser was moved from a standalone gem to something incucumber
mono repo but looks like something may not have been updated properly.Expected Behavior
Run without failure
Current Behavior
It fails to find the library/file.
Possible Solution
Temp work around add
~> 5.1
to yourGemfile
orgemspec
while waiting for a fix.Steps to Reproduce (for bugs)
I experienced this issue with running foodcritic
Your Environment
The text was updated successfully, but these errors were encountered: