Skip to content

test(benchmark): add basic sourcemap bench case#9100

Merged
h-a-n-a merged 1 commit intomainfrom
test-sourcemap
Jan 23, 2025
Merged

test(benchmark): add basic sourcemap bench case#9100
h-a-n-a merged 1 commit intomainfrom
test-sourcemap

Conversation

@h-a-n-a
Copy link
Copy Markdown
Contributor

@h-a-n-a h-a-n-a commented Jan 23, 2025

Summary

Fixed an issue of source map plugin not being added.

Added a bench case.

Checklist

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

@github-actions github-actions Bot added the team The issue/pr is created by the member of Rspack. label Jan 23, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 23, 2025

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit b2457f7
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/6791fc6f71dc0c000852e872

@h-a-n-a h-a-n-a enabled auto-merge (squash) January 23, 2025 08:24
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Jan 23, 2025

CodSpeed Performance Report

Merging #9100 will not alter performance

Comparing test-sourcemap (b2457f7) with main (9e1c427)

🎉 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

✅ 3 untouched benchmarks
🆕 1 new benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
🆕 basic_sourcemap N/A 2.8 ms N/A

@h-a-n-a h-a-n-a merged commit 9df4956 into main Jan 23, 2025
@h-a-n-a h-a-n-a deleted the test-sourcemap branch January 23, 2025 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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