-
Notifications
You must be signed in to change notification settings - Fork 224
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
Document workflow.init for Python #3108
Conversation
1e124f3
to
649e2a1
Compare
cd29aed
to
b40bb93
Compare
b40bb93
to
f77a9c7
Compare
7a93dbb
to
2b180d0
Compare
Continuous Integration will not build until /encyclopedia/workflow-message-passing#workflow-initializers is available, which is @drewhoskins-temporal 's ticket/PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moving this forward.
6aecfa0
to
e579e01
Compare
Thanks @drewhoskins-temporal pushed fixes for both of those, and I also changed the example to make it wait for the update (sample). |
@dandavison So what is the status given those fixes and updates? Merge now? Wait? Thanks |
No, this still can't be merged until next Python SDK release. I'll notify when that occurs. |
Moved to "Pending Release" column on board. |
Document
@workflow.init
, added in temporalio/sdk-python#645.Notes to reviewers: