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

Watch mode for node package executor exits immediately #5208

Closed
DibyodyutiMondal opened this issue Apr 2, 2021 · 3 comments
Closed

Watch mode for node package executor exits immediately #5208

DibyodyutiMondal opened this issue Apr 2, 2021 · 3 comments
Labels
outdated scope: node Issues related to Node, Express, NestJS support for Nx type: bug

Comments

@DibyodyutiMondal
Copy link

Current Behavior

Running the @nrwl/node:package executor in watch mode exits immediately after the initial compilation
Looking to update the dependencies, I ran npx nx migrate @nrwl/workspace
This updated @nrwl/node as well, and then the build target with watch mode stopped working as it used to

9:28:36 am - Starting compilation in watch mode...


9:28:38 am - Found 0 errors. Watching for file changes.
Copying asset files...   
Done copying asset files.
PS my/workspace/path>       <== the command has exited immediately

Expected Behavior

The command should not exit and continue watching for changes
I ran npx nx migrate @nrwl/node@11.4.0 (the version I was on before I ran the workspace update) and the problem disappeared
@nrwl/workspace and other nrwl libraries are still at v11.6.1

9:35:57 am - Starting compilation in watch mode...


9:35:59 am - Found 0 errors. Watching for file changes.
Copying asset files...   
Done copying asset files.         <=== the command continues

Environment

Node : 14.16.0
OS : win32 x64
npm : 6.14.11

nx : Not Found
@nrwl/angular : 11.6.1
@nrwl/cli : 11.6.1
@nrwl/cypress : 11.6.1
@nrwl/devkit : 11.6.1
@nrwl/eslint-plugin-nx : 11.6.1
@nrwl/express : Not Found
@nrwl/jest : 11.6.1
@nrwl/linter : 11.6.1
@nrwl/nest : Not Found
@nrwl/next : Not Found
@nrwl/node : 11.6.1
@nrwl/react : Not Found
@nrwl/schematics : Not Found
@nrwl/tao : 11.6.1
@nrwl/web : Not Found
@nrwl/workspace : 11.6.1
@nrwl/storybook : Not Found
@nrwl/gatsby : Not Found
typescript : 4.1.5

@vsavkin vsavkin added the scope: node Issues related to Node, Express, NestJS support for Nx label Apr 4, 2021
@DibyodyutiMondal
Copy link
Author

DibyodyutiMondal commented Apr 16, 2021

This still exists in @nrwl/node 12.0.6

In fact, if @nrwl/workspace is as v12, then even rolling back @nrwl/node to v11 does not work, as it did previously

EDIT: It works as long as @nrwl/node is at 11.4.0 (I was mistakenly trying 11.6.1)

bekos added a commit to bekos/nx that referenced this issue May 7, 2021
bekos added a commit to bekos/nx that referenced this issue May 7, 2021
bekos added a commit to bekos/nx that referenced this issue May 7, 2021
@DibyodyutiMondal
Copy link
Author

Can confirm it works using @nrwl/node@12.3.4

Thank you, nrwl! You are amazing!

@github-actions
Copy link

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated scope: node Issues related to Node, Express, NestJS support for Nx type: bug
Projects
None yet
Development

No branches or pull requests

2 participants