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/messages@22.0.0 #1180

Closed
3 tasks done
Chris-Belga-NF opened this issue Apr 24, 2024 · 2 comments
Closed
3 tasks done

Comments

@Chris-Belga-NF
Copy link

Chris-Belga-NF commented Apr 24, 2024

Current behavior

The cypress-cucumber-preprocessor package relies on @cucumber/messages@22.0.0, which in turn depends on an outdated version of reflect-metadata. This outdated version triggers a critical bug in fallback handling, as indicated by the warning message: "This version has a critical bug in fallback handling. Please upgrade to reflect-metadata@0.2.2 or newer."

Desired behavior

I would like the cypress-cucumber-preprocessor package to use an updated version of @cucumber/messages(24.1.0) that uses an updated version of reflect-metadata that resolves "This version has a critical bug in fallback handling. Please upgrade to reflect-metadata@0.2.2 or newer.".

Versions

  • Cypress version: cypress@13.8.1
  • Preprocessor version: @badeball/cypress-cucumber-preprocessor@20.0.4
  • Node version: 20.12.2

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).
@shieldsi
Copy link

I am also experiencing this issue using @badeball/cypress-cucumber-preprocessor@20.0.4

badeball added a commit that referenced this issue Apr 28, 2024
* Nearly all, except some packages that have migrated to ESM and are
  more diffult to update, in addition to eslint, for which
  typescript-eslint doesn't support yet [1[.

This fixes #1180 [2].

[1] typescript-eslint/typescript-eslint#8211
[2] #1180
badeball added a commit that referenced this issue Apr 28, 2024
* Nearly all, except some packages that have migrated to ESM and are
  more diffult to update, in addition to

  - eslint, for which typescript-eslint doesn't support yet [1[, and
  - prettier, for which the associated extension remains behind [2]

This fixes #1180 [3].

[1] typescript-eslint/typescript-eslint#8211
[2] prettier/prettier-vscode#3142
[3] #1180
@badeball
Copy link
Owner

Hi @Chris-Belga-NF, I've fixed this with v20.0.5.

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