Skip to content

Add workflow_task_list parameter to WorkflowTask#455

Merged
ewjoachim merged 2 commits into
mainfrom
inherit-child-decision-task-list
Jun 15, 2026
Merged

Add workflow_task_list parameter to WorkflowTask#455
ewjoachim merged 2 commits into
mainfrom
inherit-child-decision-task-list

Conversation

@ewjoachim

Copy link
Copy Markdown
Contributor

In this PR we add a workflow_task_list to WorkflowTask.
When set, child workflows will be launched in the said task list

Important

This can be a breaking change if downstream user workflows already use something named workflow_task_list.

@ewjoachim ewjoachim force-pushed the inherit-child-decision-task-list branch from 107c1d8 to c864e76 Compare June 10, 2026 17:32
Comment thread simpleflow/task.py Outdated
# precedence over the child workflow's own ``get_task_list``/``task_list``
# (see simpleflow.swf.task.WorkflowTask.task_list). It is *not* a run
# argument: it never reaches the child's ``run()``.
self._task_list: str | None = workflow_task_list

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Curious: why is typing needed here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably an artifact of how it was written. I can remove it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@ewjoachim ewjoachim force-pushed the inherit-child-decision-task-list branch from c864e76 to 79eb261 Compare June 11, 2026 14:39

@ybastide ybastide left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice 🙂
Could this be documented?

@ewjoachim

Copy link
Copy Markdown
Contributor Author

Could this be documented?

It now is, thank you !

@ybastide ybastide left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@ewjoachim ewjoachim merged commit 86fa03b into main Jun 15, 2026
9 checks passed
@ewjoachim ewjoachim deleted the inherit-child-decision-task-list branch June 15, 2026 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants