ci: fix Codspeed profiling#19866

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft

hai-x wants to merge1 commit intomain

base:main

Choose a base branch

Draft

hai-x wants to merge1 commit intomainfromcodspeed-profile

+1 −1

Conversation

Copy link

Member

What kind of change does this PR introduce?

Fixes benchmark perf profiling is unavailable inCodspeed.

I think we can revert it whenCodSpeedHQ/codspeed-node#51 is merged.

Did you add tests for your changes?

Existing

Does this PR introduce a breaking change?

What needs to be documented once your changes are merged?

ci: fix Codspeed profiling

ece6856

Copy link

_{Comparingcodspeed-profile (ece6856) withmain (2d25a95)}

⚡ 1 improvements
❌ 2 regressions
✅ 39 untouched benchmarks

⚠️Please fix the performance issues oracknowledge them on CodSpeed.

	Benchmark	`BASE`	`HEAD`	Change
❌	`benchmark "devtool-eval", scenario '{"name":"mode-development-rebuild","mode":"development","watch":true}'`	41.1 ms	123.1 ms	-66.62%
❌	`benchmark "many-chunks-esm", scenario '{"name":"mode-development-rebuild","mode":"development","watch":true}'`	101.4 ms	148.2 ms	-31.58%
⚡	`benchmark "react", scenario '{"name":"mode-development-rebuild","mode":"development","watch":true}'`	111.3 ms	81.8 ms	+36.01%