nx
nx copied to clipboard
feat(core): move target defaults back to post graph step
Current Behavior
We moved target defaults to a graph plugin around the release of Nx 18. It never fit really perfectly, but because we wanted to apply them before project.json / package.json configuration results which are plugins and do make sense as a plugin it made sense at the time.
Expected Behavior
We want to expand the behavior of target defaults in the near future to better handle pattern matching. In doing this, we need more flexibility in how its applied and that process will need to be aware of which targets are going to be created by plugins. As such, we are moving it back to being a post-step in the project configuration construction.
Related Issue(s)
Fixes #
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
| Name | Status | Preview | Updated (UTC) |
|---|---|---|---|
| nx-dev | ⬜️ Ignored (Inspect) | Visit Preview | Jun 27, 2024 6:44pm |
☁️ Nx Cloud Report
CI is running/has finished running commands for commit 77a167e953662d238dac2931d0bff1b87136d0aa. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.
📂 See all runs for this CI Pipeline Execution
✅ Successfully ran 6 targets
nx affected --targets=e2e,e2e-ci --base=f5c8eda6e10c62ed02c6b0b58b87797d0f8a0407 --head=77a167e953662d238dac2931d0bff1b87136d0aa --parallel=1nx run-many -t check-imports check-commit check-lock-files check-codeowners documentation --parallel=1 --no-dtenx affected --targets=lint,test,build --base=f5c8eda6e10c62ed02c6b0b58b87797d0f8a0407 --head=77a167e953662d238dac2931d0bff1b87136d0aa --parallel=3nx-cloud record -- nx format:check --base=f5c8eda6e10c62ed02c6b0b58b87797d0f8a0407 --head=77a167e953662d238dac2931d0bff1b87136d0aanx documentation --no-dtenx affected -t e2e-macos-ci --parallel=1 --base=f5c8eda6e10c62ed02c6b0b58b87797d0f8a0407 --head=77a167e953662d238dac2931d0bff1b87136d0aa
Sent with 💌 from NxCloud.
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.