Skip to content

[don't merge] time skipping integration#1549

Open
feiyang3cat wants to merge 1 commit into
temporalio:mainfrom
feiyang3cat:ts
Open

[don't merge] time skipping integration#1549
feiyang3cat wants to merge 1 commit into
temporalio:mainfrom
feiyang3cat:ts

Conversation

@feiyang3cat
Copy link
Copy Markdown

@feiyang3cat feiyang3cat commented May 21, 2026

What was changed

provide a user interface and 2 patterns to integrate with the new time skipping on the server side, just as a starting point for discussion

(Credit to Chad's previous PR — borrowed some designs and code from there, with updates needed since the API design has changed)

The tests fail because I added some code that use some time skipping apis that are not shipped in the dev-server of CLI yet.

@feiyang3cat feiyang3cat requested a review from a team as a code owner May 21, 2026 07:11
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@feiyang3cat feiyang3cat force-pushed the ts branch 5 times, most recently from 5ad5c1d to eb93a78 Compare May 21, 2026 22:08
task_queue=worker.task_queue,
)

# Skip 1h of virtual time; bound pauses skipping so we can interact.
Copy link
Copy Markdown
Author

@feiyang3cat feiyang3cat May 21, 2026

Choose a reason for hiding this comment

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

set the 1h directly here than in the configuration will be better

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.

2 participants