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(node): consolidate js and node plugins #9086
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/nrwl/nx-dev/4JKqA4f7st6M89yr4RdxEwVLEycN [Deployment for 1d65804 canceled] |
e5e99df
to
f80396e
Compare
f80396e
to
0eb8844
Compare
0eb8844
to
f8b1a76
Compare
f8b1a76
to
e3f3420
Compare
e3f3420
to
adf8798
Compare
adf8798
to
0294d4d
Compare
0294d4d
to
ef68ac1
Compare
ef68ac1
to
b48bb94
Compare
b48bb94
to
087590e
Compare
087590e
to
d6d31c7
Compare
d6d31c7
to
a66fcea
Compare
a66fcea
to
904b4b5
Compare
904b4b5
to
8ae6fab
Compare
8ae6fab
to
b425c22
Compare
b425c22
to
d9d7106
Compare
d9d7106
to
68e09e5
Compare
68e09e5
to
1a12c60
Compare
1a12c60
to
c9166c1
Compare
c9166c1
to
1d65804
Compare
@@ -9,13 +9,6 @@ | |||
"x-type": "library", | |||
"description": "Create a library" | |||
}, | |||
"application": { |
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 is still referenced in the docs 🙃
https://nx.dev/getting-started/nx-and-typescript#create-a-typescript-based-application
Tripped me up why this wasn't working.
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.
Propogated this message internally to see what we could do there. Thank you
* chore(node): update docs after consolidating js and node plugins * feat(node): consolidate js and node plugins
FYI: We didn't get any automatic migrations when we tried to go from 13.8.3 to 13.8.5 -- also, shouldn't this have been a major version update? |
@fbartho since we provided the migration, the changes didn't make it to a major version update. I wasn't able to reproduce this. I ran the following commands to try to reproduce:
At this point, my
Now my If you still run into issues, please open a new issue with reproduce steps so we can track it better. Thanks. |
Plugins update for nx after migration to version 14 nrwl/nx#9086
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. |
This PR consolidates the executors and generators in the
@nrwl/js
and@nrwl/node
plugins.Why?
We want to move any Node concerns back to the
@nrwl/node
plugin, same way that we have@nrwl/web
handle everything that's specific to the web platform.The
@nrwl/js
plugin is trimmed to just handle to generic TypeScript project development--no assumptions on runtime platform. You can still wire up a Node app using only@nrwl/js
, just as you could make a web app the using it.The
@nrwl/node
plugin provides additional features like static linking (i.e. bundling) that may be desirable when shipping Node apps as microservices.Note, most of the changes are in the first commit. The other changes are updating docs.
Changes
Remove
@nrwl/js:app
generator since it is no longer a use-case for the JS pluginRemove
@nrwl/js:node
executor + migration to use@nrwl/node:node
and install@nrwl/node
if necessaryRemove
@nrwl/node:package
executor since users should use JS plugin to build packages + migration to use@nrwl/js:tsc
Rename
@nrwl/node:build
executor to@nrwl/node:webpack
+ migrationRename
@nrwl/node:execute
executor to@nrwl/node:node
+ migrationUpdate
@nrwl/nx-plugin:plugin
to use@nrwl/js:library
when generating new plugin (instead of@nrwl/node:library
)Re-enables SWC e2e tests, now that CI is on Node 16