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

NPM Warn deprecated reflect-metadata@0.2.1 #1215

Open
3 tasks done
CalebCyrill opened this issue Jul 22, 2024 · 5 comments
Open
3 tasks done

NPM Warn deprecated reflect-metadata@0.2.1 #1215

CalebCyrill opened this issue Jul 22, 2024 · 5 comments

Comments

@CalebCyrill
Copy link

Current behavior

When installing the preprocessor I received the warning mesage npm WARN deprecated reflect-metadata@0.2.1: This version has a critical bug in fallback handling. Please upgrade to reflect-metadata@0.2.2 or newer.npm WARN deprecated reflect-metadata@0.2.1: This version has a critical bug in fallback handling. Please upgrade to reflect-metadata@0.2.2 or newer. A previously closed bug stated that this was handled in 20.0.5, but I am seeing it again in the latest version.

Desired behavior

The desired behavior is that it does not show the warning message that the deprecated reflect-metadata is still being used

Test code to reproduce

Versions

  • Cypress version: 13.13.0
  • Preprocessor version: 20.1.0
  • Node version: 20.9.0

Checklist

  • I've read the FAQ.
  • I've read instructions for logging issues.
  • I'm not using cypress-cucumber-preprocessor@4.3.1 (package name has changed and it is no longer the most recent version, see #689).
@badeball
Copy link
Owner

The latest @cucumber/gherkin depends on an old version of @cucumber/messages, which in turn depends on reflect-metadata@0.2.1.

cucumber/gherkin#252 fixes this, but has yet to be released.

@mikibakaiki
Copy link

I'm also getting the same warnings, with another one added:

npm warn deprecated glob@7.2.3: Glob versions prior to v9 are no longer supported
npm warn deprecated glob@8.1.0: Glob versions prior to v9 are no longer supported
npm warn deprecated reflect-metadata@0.2.1: This version has a critical bug in fallback handling. Please upgrade to reflect-metadata@0.2.2 or newer.
npm warn deprecated reflect-metadata@0.2.1: This version has a critical bug in fallback handling. Please upgrade to reflect-metadata@0.2.2 or newer.
npm warn deprecated reflect-metadata@0.2.1: This version has a critical bug in fallback handling. Please upgrade to reflect-metadata@0.2.2 or newer.

@mikibakaiki
Copy link

seems that a new version has been released 😄

@badeball
Copy link
Owner

badeball commented Sep 6, 2024

Said package isn't the only source of reflect-metadata, as seen below.

$ npm ls reflect-metadata
esbuild-ts@ /foobar/esbuild-ts
└─┬ @badeball/cypress-cucumber-preprocessor@20.1.2
  ├─┬ @cucumber/cucumber@11.0.0
  │ ├─┬ @cucumber/gherkin-utils@9.0.0
  │ │ └─┬ @cucumber/messages@24.1.0
  │ │   └── reflect-metadata@0.2.1
  │ └─┬ @cucumber/messages@24.1.0
  │   └── reflect-metadata@0.2.1
  ├─┬ @cucumber/gherkin@29.0.0
  │ └─┬ @cucumber/messages@25.0.1
  │   └── reflect-metadata@0.2.2 deduped
  └─┬ @cucumber/messages@26.0.0
    └── reflect-metadata@0.2.2

Tracking a bunch of transitive dependencies isn't something I'm going to spend my time on. If you have a request to update a specific dependency, then you can open up a ticket on this tracker. Before that, this is nothing but noise to me and not actionable.

@mikibakaiki
Copy link

Yeah, i agree it would be time consuming and not worth the effort! would you care to use RenovateBot as it would take care of most of all these things for you? :)

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

No branches or pull requests

3 participants