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

move away from reflect-metadata #13600

Closed
1 task done
cayter opened this issue May 19, 2024 · 1 comment
Closed
1 task done

move away from reflect-metadata #13600

cayter opened this issue May 19, 2024 · 1 comment
Labels
needs triage This issue has not been looked into type: enhancement 🐺

Comments

@cayter
Copy link

cayter commented May 19, 2024

Is there an existing issue that is already proposing this?

  • I have searched the existing issues

Is your feature request related to a problem? Please describe it

Screenshot 2024-05-19 at 3 42 18 PM

As mentioned in the above, tc39 has finalised the decorator metadata spec which leads to reflect-metadata implementation being outdated.

esbuild has recently released decorator metadata support that follows tc39 stage 3 spec which makes it super easy for us to run nestjs app via node --import=tsx app.ts (assuming tsx is gonna upgrade its esbuild dependency to v0.21.3 soon).

Describe the solution you'd like

N/A

Teachability, documentation, adoption, migration strategy

N/A

What is the motivation / use case for changing the behavior?

  • Much simpler setup.
  • Better parity between dev/prd.
@cayter cayter added needs triage This issue has not been looked into type: enhancement 🐺 labels May 19, 2024
@micalevisk
Copy link
Member

Duplicated #11414

@micalevisk micalevisk closed this as not planned Won't fix, can't repro, duplicate, stale May 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs triage This issue has not been looked into type: enhancement 🐺
Projects
None yet
Development

No branches or pull requests

2 participants