Skip to content

Conversation

@acroca
Copy link
Member

@acroca acroca commented Jan 16, 2026

Adds support for workflow versioning, both using patches and version names.

This implementation is almost exactly as the one in the Go SDK, both internally as well as the user-facing changes.

Requires dapr/durabletask-python#40

@cicoyle cicoyle added this to the v1.17 milestone Jan 16, 2026
@acroca acroca force-pushed the workflow-versioning branch 4 times, most recently from 35dfa08 to 8782c29 Compare January 26, 2026 12:54
Signed-off-by: Albert Callarisa <albert@diagrid.io>
@acroca acroca force-pushed the workflow-versioning branch from 8782c29 to dfacaa9 Compare January 26, 2026 13:16
@codecov
Copy link

codecov bot commented Jan 26, 2026

Codecov Report

❌ Patch coverage is 10.25641% with 35 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.52%. Comparing base (bffb749) to head (dfacaa9).
⚠️ Report is 65 commits behind head on main.

Files with missing lines Patch % Lines
...ext-workflow/dapr/ext/workflow/workflow_runtime.py 6.06% 31 Missing ⚠️
...orkflow/dapr/ext/workflow/dapr_workflow_context.py 33.33% 2 Missing ⚠️
...r-ext-workflow/dapr/ext/workflow/workflow_state.py 33.33% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #893      +/-   ##
==========================================
- Coverage   86.63%   86.52%   -0.11%     
==========================================
  Files          84      103      +19     
  Lines        4473     6939    +2466     
==========================================
+ Hits         3875     6004    +2129     
- Misses        598      935     +337     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@acroca acroca marked this pull request as ready for review January 26, 2026 13:22
@acroca acroca requested review from a team as code owners January 26, 2026 13:22
Copy link
Contributor

@elena-kolevska elena-kolevska left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you for the extra effort for a detailed example!

@acroca acroca added this pull request to the merge queue Jan 27, 2026
Merged via the queue into dapr:main with commit 39d0036 Jan 27, 2026
11 of 13 checks passed
@acroca acroca mentioned this pull request Jan 27, 2026
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.

3 participants