Skip to content

Implements DO blockConcurrencyWhile test for Python Workers.#6262

Merged
dom96 merged 1 commit intomainfrom
dominik/do-block-concurrency-while-test
Mar 6, 2026
Merged

Implements DO blockConcurrencyWhile test for Python Workers.#6262
dom96 merged 1 commit intomainfrom
dominik/do-block-concurrency-while-test

Conversation

@dom96
Copy link
Contributor

@dom96 dom96 commented Mar 5, 2026

Just some tests to ensure we have coverage for blockConcurrencyWhile

Resolves EW-9784

@dom96 dom96 requested review from hoodmane and ryanking13 March 5, 2026 18:40
@dom96 dom96 requested review from a team as code owners March 5, 2026 18:40
@ask-bonk
Copy link
Contributor

ask-bonk bot commented Mar 5, 2026

ResolveMessage: Cannot find module '@opencode-ai/plugin' from '/home/runner/work/workerd/workerd/.opencode/tools/bazel-deps.ts'

github run

@ask-bonk
Copy link
Contributor

ask-bonk bot commented Mar 5, 2026

@dom96 Bonk workflow failed. Check the logs for details.

View workflow run · To retry, trigger Bonk again.

@codspeed-hq
Copy link

codspeed-hq bot commented Mar 5, 2026

Merging this PR will degrade performance by 21.08%

❌ 1 regressed benchmark
✅ 69 untouched benchmarks
⏩ 129 skipped benchmarks1

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
bodyWithHeaders[Response] 30.4 µs 38.5 µs -21.08%

Comparing dominik/do-block-concurrency-while-test (9f1a399) with main (ea98a22)

Open in CodSpeed

Footnotes

  1. 129 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@dom96 dom96 merged commit 5e52fc0 into main Mar 6, 2026
26 of 27 checks passed
@dom96 dom96 deleted the dominik/do-block-concurrency-while-test branch March 6, 2026 12:36
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