nx
nx copied to clipboard
feat(core): add --only option to nx affected
ISSUES CLOSED: #11478
I'm not 100% sure I added the only
arg in the right place, or in the right fashion. But this was my best guess. Please let me know if anything isn't right.
Current Behavior
nx affected
cannot run on only a whitelist of projects.
Expected Behavior
nx affected --only myapp
Can now run a target on a subset of projects, only if affected.
Related Issue(s)
Fixes #11478
☁️ Nx Cloud Report
CI is running/has finished running commands for commit 0cc297a806d4e81240f0ece31b5819f86ea3b9e8. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.
📂 See all runs for this branch
✅ Successfully ran 12 targets
-
nx affected --target=e2e --base=f6338d7e53f36148c69fd310a7b0d72994682677 --head=0cc297a806d4e81240f0ece31b5819f86ea3b9e8 --exclude=e2e-storybook,e2e-storybook-angular,e2e-react-native,e2e-detox --parallel=1
-
nx affected --target=test --base=f6338d7e53f36148c69fd310a7b0d72994682677 --head=0cc297a806d4e81240f0ece31b5819f86ea3b9e8 --parallel=1
-
nx affected --target=lint --base=f6338d7e53f36148c69fd310a7b0d72994682677 --head=0cc297a806d4e81240f0ece31b5819f86ea3b9e8 --parallel=3
-
nx affected --target=build --base=f6338d7e53f36148c69fd310a7b0d72994682677 --head=0cc297a806d4e81240f0ece31b5819f86ea3b9e8 --parallel=3
-
nx-cloud record -- yarn nx workspace-lint
-
nx-cloud record -- yarn documentation
-
nx build typedoc-theme
-
nx-cloud record -- yarn depcheck
-
nx-cloud record -- yarn check-lock-files
-
nx-cloud record -- yarn check-commit
-
nx-cloud record -- yarn nx format:check --base=f6338d7e53f36148c69fd310a7b0d72994682677 --head=0cc297a806d4e81240f0ece31b5819f86ea3b9e8
-
nx-cloud record -- yarn check-imports
Sent with 💌 from NxCloud.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Updated |
---|---|---|---|
nx-dev | ✅ Ready (Inspect) | Visit Preview | Aug 7, 2022 at 7:25AM (UTC) |
Hey @AlexJWayne! This PR has gone a while without comment from us, unfortunately. Looking at this now, I'm not sure that we would want to add this flag, and feel like we need some more discussion before moving forward.
For the time being I'm going to close this, in favor of discussing further in #11478. After we agree on the path forward, a new PR can be drafted.
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.