Skip to content

Conversation

@dandavison
Copy link
Contributor

@dandavison dandavison commented Jul 14, 2025

This is PR 5/5 in a stack.

This is a small cleanup. Currently the get_operation_factory method is used by the Temporal SDK (vendored, since it is not exposed). This commit will allow the Temporal SDK to use get_operation_definition which is exposed.

@dandavison dandavison force-pushed the operation-definition branch from 4fa4f0e to 79c228f Compare July 14, 2025 13:04
@dandavison dandavison force-pushed the get-operation-factory branch from 3154c44 to 9b5e8d2 Compare July 14, 2025 13:04
@dandavison dandavison force-pushed the operation-definition branch from 79c228f to 562a2fd Compare July 14, 2025 15:02
@dandavison dandavison force-pushed the get-operation-factory branch from 9b5e8d2 to 0de1328 Compare July 14, 2025 15:02
@dandavison dandavison force-pushed the operation-definition branch from 562a2fd to 36da5d3 Compare July 14, 2025 15:08
@dandavison dandavison force-pushed the get-operation-factory branch from 0de1328 to 31a187e Compare July 14, 2025 15:08
@dandavison dandavison force-pushed the operation-definition branch from 36da5d3 to 9db6a66 Compare July 14, 2025 15:56
@dandavison dandavison force-pushed the get-operation-factory branch from 31a187e to ace2470 Compare July 14, 2025 15:56
@dandavison dandavison force-pushed the operation-definition branch from 9db6a66 to b6a3d13 Compare July 14, 2025 17:42
@dandavison dandavison force-pushed the get-operation-factory branch from ace2470 to ba6b3ad Compare July 14, 2025 17:42
Base automatically changed from operation-definition to main July 14, 2025 19:18
AI: This commit should have tag name get-operation-factory and should be
force-pushed to a branch of that name as necessary.
@dandavison dandavison force-pushed the get-operation-factory branch from ba6b3ad to eb45d15 Compare July 14, 2025 19:20
@dandavison dandavison merged commit 2f9f2c0 into main Jul 14, 2025
10 checks passed
@dandavison dandavison deleted the get-operation-factory branch July 14, 2025 19:26
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