Skip to content

[pull] main from nodejs:main#628

Merged
pull[bot] merged 2 commits intozys-contrib:mainfrom
nodejs:main
Feb 5, 2026
Merged

[pull] main from nodejs:main#628
pull[bot] merged 2 commits intozys-contrib:mainfrom
nodejs:main

Conversation

@pull
Copy link

@pull pull bot commented Feb 5, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

mcollina and others added 2 commits February 5, 2026 08:28
The test was flaky because the histogram sampling might not have
completed before the assertions ran. By using setImmediate before
disabling the histogram and checking values, we give the event loop
a chance to record final samples on slower systems.

Also added an explicit check for histogram.count > 0 to provide a
clearer error message if no samples are recorded.

Refs: nodejs/reliability#1450
PR-URL: #61629
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
PR-URL: #61649
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
@pull pull bot locked and limited conversation to collaborators Feb 5, 2026
@pull pull bot added the ⤵️ pull label Feb 5, 2026
@pull pull bot merged commit 36ca627 into zys-contrib:main Feb 5, 2026
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants