Skip to content

Add workflow cancellation reason#1574

Merged
tconley1428 merged 4 commits into
mainfrom
add-wf-cancel-reason
Jun 3, 2026
Merged

Add workflow cancellation reason#1574
tconley1428 merged 4 commits into
mainfrom
add-wf-cancel-reason

Conversation

@Sushisource
Copy link
Copy Markdown
Member

@Sushisource Sushisource commented Jun 2, 2026

What was changed

Added method for getting workflow cancellation reason

Why?

Parity gap with other SDKs

Checklist

  1. Closes Expose Workflow cancel cause/reason #764

  2. How was this tested:

  1. Any docs updates needed?

@Sushisource Sushisource requested a review from a team as a code owner June 2, 2026 23:03
@Sushisource Sushisource force-pushed the add-wf-cancel-reason branch from f19987e to 48cefca Compare June 2, 2026 23:06
Comment thread temporalio/worker/_workflow_instance.py
Copy link
Copy Markdown
Contributor

@mjameswh mjameswh left a comment

Choose a reason for hiding this comment

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

Approving, but please double-check the assumption that we'll always have a cancellation reason.

@tconley1428 tconley1428 merged commit b133ed9 into main Jun 3, 2026
15 checks passed
@tconley1428 tconley1428 deleted the add-wf-cancel-reason branch June 3, 2026 16:39
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.

Expose Workflow cancel cause/reason

4 participants