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

Before/After __all__ directives and allow for multiple tasks #76

Open
ryanrichholt opened this issue Jul 23, 2019 · 0 comments
Open

Before/After __all__ directives and allow for multiple tasks #76

ryanrichholt opened this issue Jul 23, 2019 · 0 comments
Labels
enhancement New feature or request

Comments

@ryanrichholt
Copy link
Collaborator

ryanrichholt commented Jul 23, 2019

Defining a task which runs "after everything else is done" can be accomplished now by using the regex flow directives: after-re: .*, before-re: .* This works by matching all other task names, and implicit removal of self-dependency directives in the workflow graph constructor.

The improvement would allow a keyword, maybe __all__, to be used with the standard flow directives. This would also allow multiple tasks to be declared as "pre" and "post" tasks. I'm still not sure how the execution order should work. I think it would be great if they fired in the same order that they were declared, but I will need to review the code and test to make sure the order is preserved when saving/modifying workflows.

@ryanrichholt ryanrichholt added the enhancement New feature or request label Jul 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant