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

Dependency on Deprecated Version of reflect-metadata in @cucumber/[email protected] #2457

Closed
swc opened this issue Dec 19, 2024 · 1 comment · Fixed by #2466
Closed

Dependency on Deprecated Version of reflect-metadata in @cucumber/[email protected] #2457

swc opened this issue Dec 19, 2024 · 1 comment · Fixed by #2466
Labels
🏦 debt Tech debt ✅ accepted The core team has agreed that it is a good idea to fix this

Comments

@swc
Copy link

swc commented Dec 19, 2024

👓 What did you see?

The current dependency on cucumber/[email protected] depends on a deprecated version of reflect-metadata:

npm warn deprecated [email protected]: This version has a critical bug in fallback handling. Please upgrade to [email protected] or newer.

✅ What did you expect to see?

The dependency was updated in cucumber/[email protected]. Can someone please update to at least this version?

📦 Which tool/library version are you using?

No response

🔬 How could we reproduce it?

No response

📚 Any additional context?

No response

@davidjgoss davidjgoss added ✅ accepted The core team has agreed that it is a good idea to fix this 🏦 debt Tech debt labels Jan 7, 2025
@swc
Copy link
Author

swc commented Jan 15, 2025

This has been done according to the original ticket request (thanks!) but I do see there is one more transitive dependency on [email protected] coming in via cucumber/gherkin-utils. gherkin-utils itself will need to be updated. I opened an issue there:

cucumber/gherkin-utils#97

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏦 debt Tech debt ✅ accepted The core team has agreed that it is a good idea to fix this
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants