Skip to content

Exclude tasks that should not run from the dep graph #865

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dobesv
Copy link

@dobesv dobesv commented Mar 17, 2025

If a task should not run, it shouldn't generate dependencies, either.

For example if we have a task "bundle" which depends on "build", running the "bundle" task should not run "build" in workspaces that do not have a "bundle" task in them.

@dobesv dobesv requested a review from kenotron as a code owner March 17, 2025 07:58
@dobesv
Copy link
Author

dobesv commented Mar 17, 2025

Note: I haven't been able to run the tests yet, I ran into some problems that I opened an issue about. So this hasn't updated the tests, yet. But I thought I could open the PR to get feedback on the approach anyway. Just in case this needs to be re-written. But in any case I'll have to figure out the issue with the tests.

@dobesv
Copy link
Author

dobesv commented Mar 17, 2025

I'm not sure this covers all the cases I want. For example, it would be nice to have type: 'worker' and still only have that generate deps if there's some script. I can probably achieve that by implementing a custom shouldRun on the task, though.

dobesv added 2 commits April 18, 2025 21:14
If a task should not run, it shouldn't generate dependencies, either.

For example if we have a task "bundle" which depends on "build", running the "bundle" task should not run "build" in workspaces that do not have a "bundle" task in them.
@dobesv dobesv force-pushed the exclude-nonrunning-deps branch from ad6882f to 35f293c Compare April 19, 2025 04:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant