Skip to content
This repository was archived by the owner on Jun 10, 2026. It is now read-only.

fix(testing-sdk): remove duplicate store update, use invoke#86

Merged
wangyb-A merged 1 commit into
mainfrom
use_invoke
Oct 30, 2025
Merged

fix(testing-sdk): remove duplicate store update, use invoke#86
wangyb-A merged 1 commit into
mainfrom
use_invoke

Conversation

@wangyb-A

@wangyb-A wangyb-A commented Oct 29, 2025

Copy link
Copy Markdown
Contributor

Issue #, if available:

Description of changes:

  • remove duplicate store update when validating invocation response, the store is updated in _complete_workflow()
  • use client.invoke instead of invoke20150331

Dependencies

If this PR requires testing against a specific branch of the Python Language SDK (e.g., for unreleased changes), uncomment and specify the branch below. Otherwise, leave commented to use the main branch.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

- remove duplicate store update when validating invocation response
- use client.invoke instead of invoke20150331
@wangyb-A wangyb-A marked this pull request as ready for review October 29, 2025 22:59
@wangyb-A wangyb-A requested a review from yaythomas October 29, 2025 22:59
@wangyb-A

Copy link
Copy Markdown
Contributor Author

https://d2fb3uhq9167rr.cloudfront.net/index.html

python examples failed because the server is down rn :(

@yaythomas yaythomas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

:shipit:

@wangyb-A wangyb-A merged commit 6f5e776 into main Oct 30, 2025
9 of 45 checks passed
@wangyb-A wangyb-A deleted the use_invoke branch October 30, 2025 17:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants