Skip to content

fix(core): enable tracing for hook by default except in make stage#10141

Merged
hardfist merged 5 commits intomainfrom
yj/enable-tracing-by-default
Apr 23, 2025
Merged

fix(core): enable tracing for hook by default except in make stage#10141
hardfist merged 5 commits intomainfrom
yj/enable-tracing-by-default

Conversation

@hardfist
Copy link
Contributor

@hardfist hardfist commented Apr 23, 2025

Summary

enable tracing for all hook(except for hook in make stage) by default, make related hook will be called tons of time for large project which will cause large trace.json and perfeto UI unreadable so it's disabled by default.

  • after
    image

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@github-actions github-actions bot added team The issue/pr is created by the member of Rspack. release: bug fix release: bug related release(mr only) labels Apr 23, 2025
@netlify
Copy link

netlify bot commented Apr 23, 2025

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit 0e64fe7
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/6808c0e11a64fb0008c9c0e0

ahabhgk
ahabhgk previously approved these changes Apr 23, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Apr 23, 2025

CodSpeed Performance Report

Merging #10141 will not alter performance

Comparing yj/enable-tracing-by-default (0e64fe7) with main (cc20f1a)

Summary

✅ 11 untouched benchmarks

@hardfist hardfist enabled auto-merge (squash) April 23, 2025 10:51
@hardfist hardfist merged commit 189928f into main Apr 23, 2025
30 checks passed
@hardfist hardfist deleted the yj/enable-tracing-by-default branch April 23, 2025 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: bug fix release: bug related release(mr only) team The issue/pr is created by the member of Rspack.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants