fix(core): make watch command work with all and initialRun specified#32282
fix(core): make watch command work with all and initialRun specified#32282jaysoo merged 1 commit intonrwl:masterfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Hi, @FrozenPandaz |
7d29024 to
1dff491
Compare
✅ Deploy Preview for nx-docs canceled.
|
✅ Deploy Preview for nx-dev canceled.
|
|
View your CI Pipeline Execution ↗ for commit e8808dd
☁️ Nx Cloud last updated this comment at |
There was a problem hiding this comment.
Nx Cloud has identified a flaky task in your failed CI:
Since the failure was identified as flaky, the solution is to rerun CI. Because this branch comes from a fork, it is not possible for us to push directly, but you can rerun by pushing an empty commit:
git commit --allow-empty -m "chore: trigger rerun"
git push
🎓 Learn more about Self-Healing CI on nx.dev
1dff491 to
e8808dd
Compare
…32282) <!-- Please make sure you have read the submission guidelines before posting an PR --> <!-- https://github.com/nrwl/nx/blob/master/CONTRIBUTING.md#-submitting-a-pr --> <!-- Please make sure that your commit message follows our format --> <!-- Example: `fix(nx): must begin with lowercase` --> <!-- If this is a particularly complex change or feature addition, you can request a dedicated Nx release for this pull request branch. Mention someone from the Nx team or the `@nrwl/nx-pipelines-reviewers` and they will confirm if the PR warrants its own release for testing purposes, and generate it for you if appropriate. --> ## Current Behavior <!-- This is the behavior we have today --> If you specify both the `all` and `initialRun` options when running `nx watch`, `initialRun` have no effect. ## Expected Behavior <!-- This is the behavior we should expect with the changes in this PR --> The command should be called once at the beginning even if there are no file changes. ## Related Issue(s) <!-- Please link the issue being fixed so it gets closed when this is merged. --> Fixes #32281 (cherry picked from commit c364312)
…32282) <!-- Please make sure you have read the submission guidelines before posting an PR --> <!-- https://github.com/nrwl/nx/blob/master/CONTRIBUTING.md#-submitting-a-pr --> <!-- Please make sure that your commit message follows our format --> <!-- Example: `fix(nx): must begin with lowercase` --> <!-- If this is a particularly complex change or feature addition, you can request a dedicated Nx release for this pull request branch. Mention someone from the Nx team or the `@nrwl/nx-pipelines-reviewers` and they will confirm if the PR warrants its own release for testing purposes, and generate it for you if appropriate. --> ## Current Behavior <!-- This is the behavior we have today --> If you specify both the `all` and `initialRun` options when running `nx watch`, `initialRun` have no effect. ## Expected Behavior <!-- This is the behavior we should expect with the changes in this PR --> The command should be called once at the beginning even if there are no file changes. ## Related Issue(s) <!-- Please link the issue being fixed so it gets closed when this is merged. --> Fixes #32281 (cherry picked from commit c364312)
|
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
If you specify both the
allandinitialRunoptions when runningnx watch,initialRunhave no effect.Expected Behavior
The command should be called once at the beginning even if there are no file changes.
Related Issue(s)
Fixes #32281