Skip to content

Conversation

@jasnell
Copy link
Collaborator

@jasnell jasnell commented Oct 29, 2025

Extracted from #5396

Not all of the bench changes in that PR can be extracted here.

@jasnell jasnell requested review from a team as code owners October 29, 2025 00:22
@codspeed-hq
Copy link

codspeed-hq bot commented Oct 29, 2025

CodSpeed Performance Report

Merging #5429 will improve performances by 53.25%

Comparing jasnell/some-bench-updates (d8b91aa) with main (e5ace24)

Summary

⚡ 2 improvements
✅ 31 untouched
🆕 1 new
⏩ 9 skipped1

Benchmarks breakdown

Benchmark BASE HEAD Change
🆕 set_append[ApiHeaders] N/A 15.9 ms N/A
bodyWithHeaders[Response] 26.2 µs 24.1 µs +8.92%
jsonResponse[Response] 53.2 µs 34.7 µs +53.25%

Footnotes

  1. 9 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.

@anonrig anonrig merged commit 153c3ef into main Oct 29, 2025
21 checks passed
@anonrig anonrig deleted the jasnell/some-bench-updates branch October 29, 2025 02:59
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