Skip to content

fix: optimize circular-dependency-check log#9776

Merged
chenjiahan merged 7 commits intomainfrom
perf-log-display
Mar 24, 2025
Merged

fix: optimize circular-dependency-check log#9776
chenjiahan merged 7 commits intomainfrom
perf-log-display

Conversation

@fireairforce
Copy link
Contributor

@fireairforce fireairforce commented Mar 24, 2025 •
edited
Loading

Summary

I just optimize the log output of circular-dependency-rspack-plugin.

Before:

image

After:

image

Checklist

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

@github-actions github-actions bot added the release: bug fix release: bug related release(mr only) label Mar 24, 2025
@netlify
Copy link

netlify bot commented Mar 24, 2025 •
edited
Loading

✅ Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit cad08bd
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/67e158c3a490ac0007ae51f5

@codspeed-hq
Copy link

codspeed-hq bot commented Mar 24, 2025 •
edited
Loading

CodSpeed Performance Report

Merging #9776 will not alter performance

Comparing perf-log-display (cad08bd) with main (71a1834)

🎉 Hooray! codspeed-rust just leveled up to 2.7.2!

A heads-up, this is a breaking change and it might affect your current performance baseline a bit. But here's the exciting part - it's packed with new, cool features and promises improved result stability 🥳!
Curious about what's new? Visit our releases page to delve into all the awesome details about this new version.

Summary

✅ 8 untouched benchmarks

@fireairforce fireairforce marked this pull request as ready for review March 24, 2025 10:06
chenjiahan
chenjiahan reviewed Mar 24, 2025
View reviewed changes
chenjiahan
chenjiahan approved these changes Mar 24, 2025
View reviewed changes
Copy link
Member

@chenjiahan chenjiahan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@chenjiahan chenjiahan merged commit ea04ad7 into main Mar 24, 2025
31 checks passed
@chenjiahan chenjiahan deleted the perf-log-display branch March 24, 2025 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chenjiahan chenjiahan chenjiahan approved these changes

Assignees

No one assigned

Labels

release: bug fix release: bug related release(mr only)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@fireairforce @chenjiahan