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
fix(nx): calculate dependencies of other targets #9942
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
This would fix #10028, if it is correct + approved. |
Thanks for the contribution! LGTM 🎉 |
@AgentEnder It turns out that this only partially fixed #10028. It took care of the situation where there are two targetDependencies and dependent projects have both of them. Before this, only the first dependency would run. What this does not address is the situation where a dependent project only has the second of the two dependencies. In that situation, that second dependency still doesn't run. In our (fairly extensive) monorepo, this is a common scenario. I've updated the repo at https://github.com/arabull/nx-deps-not-running to reflect this. That repo offers a very simple reproduction of the problem. What is the proper protocol? Should I file a new issue for this, or does #10028 need to be reopened? |
nrwl/nx#9942 brandingbrand-source-id: f4904b30ffbfd3d0847b0a732f95f1b3099fe6c7
nrwl/nx#9942 brandingbrand-source-id: f4904b30ffbfd3d0847b0a732f95f1b3099fe6c7
nrwl/nx#9942 brandingbrand-source-id: f4904b30ffbfd3d0847b0a732f95f1b3099fe6c7
nrwl/nx#9942 brandingbrand-source-id: f4904b30ffbfd3d0847b0a732f95f1b3099fe6c7
nrwl/nx#9942 brandingbrand-source-id: f4904b30ffbfd3d0847b0a732f95f1b3099fe6c7
nrwl/nx#9942 brandingbrand-source-id: f4904b30ffbfd3d0847b0a732f95f1b3099fe6c7
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
The task list generated does not include all task for targets when the depends on includes multiple targets ie
This is a regression since 13.4.6 which I upgraded from when discovering this issue.
Expected Behavior
The task generated should include all task from all of the targets in the depends on.
Fixes: #10028