Skip to content
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

exclude_dependencies should be recursive #3822

Open
yhakbar opened this issue Jan 30, 2025 · 0 comments
Open

exclude_dependencies should be recursive #3822

yhakbar opened this issue Jan 30, 2025 · 0 comments
Labels
bug Something isn't working preserved Preserved issues never go stale

Comments

@yhakbar
Copy link
Collaborator

yhakbar commented Jan 30, 2025

Describe the bug

When a user uses the exclude_dependencies attribute of the exclude block, all ancestor dependencies should be excluded from the run queue in addition to direct dependencies.

The workaround right now is to set the same exclude block on all dependencies in the chain, but it is a more convenient feature to do this automatically. It is also likely what users expect to happen.

Expected behavior

The exclude_dependencies attribute recursively removes ancestral dependencies from the run queue.

Additional context

Avoid working on this until after #3629 . We will likely take advantage of those data structures to implement this behavior.

@yhakbar yhakbar added bug Something isn't working preserved Preserved issues never go stale labels Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working preserved Preserved issues never go stale
Projects
None yet
Development

No branches or pull requests

1 participant