-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
@nrwl/webpack with swc doesn't work with reflect-metadata/emitDecoratorMetadata #12845
Comments
I wonder if this is related to #12782? |
@MarcieMarc425 Hmm, I don't think so. With |
This issue has been automatically marked as stale because it hasn't had any recent activity. It will be closed in 14 days if no further activity occurs. |
Still an issue, I have checked with recent version. Interestingly enough @nrwl/rspack works fine with SWC. |
i am facing the same issue, can someone guide? |
This issue has been automatically marked as stale because it hasn't had any recent activity. It will be closed in 14 days if no further activity occurs. |
Haven't checked for the long time but I don't think it is resolved. |
Checked it on latest and it is resolved. See the demo here: https://github.com/jaysoo/issue12845 |
This issue has been closed for more than 30 days. If this issue is still occuring, please open a new issue with more recent context. |
main.ts
dep.ts
Current Behavior
✅ Running
@nrwl/webpack:webpack
with"compiler": "tsc"
includesDepClass
in bundle.❌ Running
@nrwl/webpack:webpack
with"compiler": "swc"
excludesDepClass
from bundle.Expected Behavior
DepClass
should be included in bundle.According to swc-project/swc#3459 this should be fixed on the side of swc already.
Steps to Reproduce
PR example: nrwl/nx-examples#240
Run
yarn nx run di:build:tsc
- ✅DepClass
is included.yarn nx run di:build:swc
- ❌DepClass
is NOT included.Environment
Related issues
The text was updated successfully, but these errors were encountered: