Skip to content

Support async sleep for sync fn-to-retry - #551

Merged
mergify[bot] merged 1 commit into
jd:mainfrom
Zac-HD:sync-fn-async-sleep
Feb 5, 2026
Merged

mergify[bot] merged 1 commit into
jd:mainfrom
Zac-HD:sync-fn-async-sleep

Conversation

@Zac-HD

@Zac-HD Zac-HD commented Feb 4, 2026 •

Copy link
Copy Markdown
Contributor

This enables us to avoid blocking the async runloop, without needing to trivially wrap the function-to-retry.

Unfortunately the relevant introspection functions are new in Python 3.10; since Python 3.9 is EOL I've also dropped it from metadata and CI configs.

@Zac-HD
Zac-HD force-pushed the sync-fn-async-sleep branch from a66b921 to d9275be Compare February 4, 2026 22:57
jd
jd previously requested changes Feb 5, 2026

@jd jd left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Would be neat to have 2 different changes

@mergify
mergify Bot dismissed jd’s stale review February 5, 2026 06:00

Pull request has been modified.

@Zac-HD
Zac-HD force-pushed the sync-fn-async-sleep branch 3 times, most recently from 9771035 to 6baad9e Compare February 5, 2026 06:19
@jd
jd force-pushed the sync-fn-async-sleep branch from 6baad9e to 1f812e7 Compare February 5, 2026 06:30
@mergify
mergify Bot merged commit 24415eb into jd:main Feb 5, 2026
9 checks passed
@mergify

mergify Bot commented Feb 5, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

✅ The pull request has been merged at 1f812e7

This pull request spent 3 seconds in the queue, with no time running CI.
The checks were run in-place.

Required conditions to merge

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