search

nrwl / nx

Smart Monorepos · Fast CI
https://nx.dev
MIT License
23.78k stars 2.37k forks source link

fix(core): create different dummy tasks for different parent tasks an… #28923

Closed FrozenPandaz closed 1 week ago

FrozenPandaz commented 1 week ago

…d dependency targets

Current Behavior

Dummy tasks are given an ID containing the parent task's target. Multiple parent tasks can create the same dummy task which overrides other dummy tasks.

Expected Behavior

Dummy tasks are given an ID containing the dependency and the parent tasks's name. This means different parent tasks will create different dummy tasks instead of overriding other ones.

Related Issue(s)

Fixes #

vercel[bot] commented 1 week ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment | Name | Status | Preview | Updated (UTC) | | :--- | :----- | :------ | :------ | | **nx-dev** | ⬜️ Ignored ([Inspect](https://vercel.com/nrwl/nx-dev/9GjasqwdLxC5HyYF9fR66MoUPbBN)) | | Nov 13, 2024 4:06pm |
nx-cloud[bot] commented 1 week ago

☁️ Nx Cloud Report

CI is running/has finished running commands for commit bd4a68e20f3297c938154212c71a82b35a41153b. 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 4 targets - [`nx affected --targets=lint,test,build,e2e,e2e-ci --base=dbaeb861023f2cd771ab3241ba9da059309e1fb2 --head=bd4a68e20f3297c938154212c71a82b35a41153b --parallel=3`](https://staging.nx.app/runs/r4RfpUgKlV?utm_source=pull-request&utm_medium=comment) - [`nx run-many -t check-imports check-commit check-lock-files check-codeowners documentation --parallel=1 --no-dte`](https://staging.nx.app/runs/Undr1NYEdO?utm_source=pull-request&utm_medium=comment) - [`nx-cloud record -- nx format:check --base=dbaeb861023f2cd771ab3241ba9da059309e1fb2 --head=bd4a68e20f3297c938154212c71a82b35a41153b`](https://staging.nx.app/runs/Hnl4CBAp2w?utm_source=pull-request&utm_medium=comment) - [`nx documentation --no-dte`](https://staging.nx.app/runs/ItMZyrIa0X?utm_source=pull-request&utm_medium=comment)

Sent with 💌 from NxCloud.

github-actions[bot] commented 5 days ago

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.