Skip to content

Refactor: Add context parameter to WorkflowRegistryV2Client methods#450

Merged
anirudhwarrier merged 3 commits into
mainfrom
DEVSVCS-4960/wrc-context
May 22, 2026
Merged

Refactor: Add context parameter to WorkflowRegistryV2Client methods#450
anirudhwarrier merged 3 commits into
mainfrom
DEVSVCS-4960/wrc-context

Conversation

@anirudhwarrier
Copy link
Copy Markdown
Collaborator

DEVSVCS-4960

This update introduces a context parameter to various methods in the WorkflowRegistryV2Client and related handlers, enhancing the ability to manage request lifetimes and cancellations. The changes include updates to transaction execution, owner linking, and workflow management functions, ensuring that context is consistently passed through the call stack. This refactor improves the overall robustness and responsiveness of the client interactions.

This update introduces a context parameter to various methods in the WorkflowRegistryV2Client and related handlers, enhancing the ability to manage request lifetimes and cancellations. The changes include updates to transaction execution, owner linking, and workflow management functions, ensuring that context is consistently passed through the call stack. This refactor improves the overall robustness and responsiveness of the client interactions.
@anirudhwarrier anirudhwarrier marked this pull request as ready for review May 22, 2026 08:35
@anirudhwarrier anirudhwarrier requested a review from a team as a code owner May 22, 2026 08:35
j-nowak
j-nowak previously approved these changes May 22, 2026
timothyF95
timothyF95 previously approved these changes May 22, 2026
@anirudhwarrier anirudhwarrier enabled auto-merge May 22, 2026 14:08
@anirudhwarrier anirudhwarrier added this pull request to the merge queue May 22, 2026
Merged via the queue into main with commit 31b81c8 May 22, 2026
22 checks passed
@anirudhwarrier anirudhwarrier deleted the DEVSVCS-4960/wrc-context branch May 22, 2026 15:13
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