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

Fixed the WorkflowExecutor and DoTaskExecutor to ignore the flow directive of skipped tasks #406

Merged
merged 1 commit into from
Sep 11, 2024

Conversation

cdavernas
Copy link
Member

Many thanks for submitting your Pull Request ❤️!

What this PR does / why we need it:

  • Fixes the WorkflowExecutor and DoTaskExecutor to ignore the flow directive of skipped tasks

…the flow directive of skipped tasks

Signed-off-by: Charles d'Avernas <charles.davernas@neuroglia.io>
@cdavernas cdavernas added priority: high Indicates a high priority issue weight: 1 An issue that has a very low development impact app: runner Concerns the Runner type: fix labels Sep 11, 2024
@cdavernas cdavernas added this to the v1.0.0 milestone Sep 11, 2024
@cdavernas cdavernas self-assigned this Sep 11, 2024
@cdavernas cdavernas merged commit 761bc43 into main Sep 11, 2024
1 check passed
@cdavernas cdavernas deleted the fix-ignore-skipped-task-directive branch September 11, 2024 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app: runner Concerns the Runner priority: high Indicates a high priority issue type: fix weight: 1 An issue that has a very low development impact
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant