-
-
Notifications
You must be signed in to change notification settings - Fork 31.9k
Add SourceMap Benchmark #58125
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Add SourceMap Benchmark #58125
Conversation
Review requested:
|
5e438af
to
4147ff3
Compare
You need to add a test for these benchmarks. You can find similar ones in test folder. |
Thank you, just pushed that! |
5749fce
to
bc9fbe9
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #58125 +/- ##
==========================================
+ Coverage 90.02% 90.12% +0.09%
==========================================
Files 639 640 +1
Lines 188173 188294 +121
Branches 36831 36927 +96
==========================================
+ Hits 169404 169701 +297
+ Misses 11509 11317 -192
- Partials 7260 7276 +16 🚀 New features to boost your workflow:
|
f1040cd
to
421ae12
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the change! Overall LGTM % nits.
02b5acc
to
7eab302
Compare
c7232d1
to
ac7a40e
Compare
Add SourceMap Benchmark
This PR introduces a new benchmark suite for the
lib/internal/source_map
module to help evaluate and explore the performance characteristics of theSourceMap
.I'm still learning how everything fits together, but this is a first step toward understanding where small improvements could improve performance.
cc @BridgeAR