-
Notifications
You must be signed in to change notification settings - Fork 40
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
feat: add emitDecoratorMetadata
compiler option
#217
Conversation
isolatedModules, experimentalDecorators, emitDecoratorMetadata
😸 |
emitDecoratorMetadata
compiler option
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Gaubee! I wanted to merge this one in because I might do a release soon, so I updated this PR to make the emitDecoratorMetadata
compiler option false
by default and removed the isolatedModules and experimentalDecorators options because those are always true
in Deno. What's the rationale behind adding these options? Do you need them?
by the way. I write my packages use deno as my toolchain for nodejs/web/deno, I couldn't find a better toolchain then this one. |
|
#226 rebase and open again |
isolatedModules, experimentalDecorators, emitDecoratorMetadata