-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
fix(js): .swcrc path should be customizable #10118
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
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.
Looks great, one small nit that will likely require updating a few unit tests.
packages/js/src/utils/schema.d.ts
Outdated
@@ -35,6 +35,7 @@ export interface ExecutorOptions { | |||
main: string; | |||
outputPath: string; | |||
tsConfig: string; | |||
swcrcPath?: string; |
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.
This prop should be required since it has a default value, and the function would not work correctly if its null.
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.
Gotcha! I've pushed the change. Unit tests pass locally without any changes. I didn't mange to get E2E tests running on main and couldn't find any coverage for this codepath so I'll just see if the tests pass on CI.
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
Current Behavior
It is currently not possible to provide a custom path to an swcrc path. Instead, JS projects using SWC must have a
.lib.swcrc
file in the project root directory.Expected Behavior
With these changes, the
@nrwl/js:swc
executor can optionally be be passed answcrcPath
.In
project.json
:Related Issue(s)
Fixes #10058