Skip to content

Refactor workflow.py into package#1488

Draft
tconley1428 wants to merge 3 commits intomainfrom
refactor-workflow-module
Draft

Refactor workflow.py into package#1488
tconley1428 wants to merge 3 commits intomainfrom
refactor-workflow-module

Conversation

@tconley1428
Copy link
Copy Markdown
Contributor

Summary

  • replace temporalio/workflow.py with a temporalio/workflow/ package split into coarse internal modules
  • preserve the temporalio.workflow package surface for existing public and SDK-private imports
  • move signal/query/update decorators and handler definition types into _handlers.py, keeping _definition.py focused on workflow class definitions

Testing

  • env UV_CACHE_DIR=/tmp/uv-cache uv run ruff check temporalio/workflow
  • env UV_CACHE_DIR=/tmp/uv-cache uv run -m pytest -p no:rerunfailures tests/test_workflow.py -q
  • env UV_CACHE_DIR=/tmp/uv-cache uv run -m pytest -p no:rerunfailures tests/test_runtime.py -q -k "not different_runtimes and not runtime_task_fail_log_forwarding and not prometheus_histogram_bucket_overrides"

Notes

  • left unrelated local uv.lock changes unstaged and out of this PR
  • did not run the 3 excluded tests/test_runtime.py cases because they require launching a local Temporal dev server in this sandbox

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.

1 participant