You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
i've been getting renovate PRs for the latest release (v8.1.0). when running npm ls to ensure that all peer dependency requirements are met (i run this as part of my standard verification for most projects), it fails reporting:
npm ERR! peer dep missing: @cucumber/message-streams@>=4.0.0, required by @cucumber/gherkin-streams@5.0.1
npm ls should exit with an exit code of zero and not report missing peer dependencies. ideally the @cucumber/cucumber package should declare a direct dependency on the @cucumber/message-streams package so that the peer requirement is not passed along to consumers of @cucumber/cucumber
…reams peer requirement for consumers (#2006)
* fix(message-streams): defined direct dependency to satisfy gherkin-streams peer requirement for consumers
closes#2005
* docs(changelog): mentioned satisfying of peer requirement by depending on massage-streams
* Update CHANGELOG.md
* Update dependency-lint.yml
* Update dependency-lint.yml
Co-authored-by: David Goss <david@davidgoss.co>
👓 What did you see?
i've been getting renovate PRs for the latest release (v8.1.0). when running
npm ls
to ensure that all peer dependency requirements are met (i run this as part of my standard verification for most projects), it fails reporting:(example failure)
✅ What did you expect to see?
npm ls
should exit with an exit code of zero and not report missing peer dependencies. ideally the@cucumber/cucumber
package should declare a direct dependency on the@cucumber/message-streams
package so that the peer requirement is not passed along to consumers of@cucumber/cucumber
📦 Which tool/library version are you using?
@cucumber/cucumber
v8.1.0🔬 How could we reproduce it?
npm install
and run verification withnpm test
📚 Any additional context?
This text was originally generated from a template, then edited by hand. You can modify the template here.
The text was updated successfully, but these errors were encountered: