Skip to content

Comments

chore: remove unused track_caller#5399

Closed
IWANABETHATGUY wants to merge 2 commits intomainfrom
07-22-chore_remove_unused_track_caller
Closed

chore: remove unused track_caller#5399
IWANABETHATGUY wants to merge 2 commits intomainfrom
07-22-chore_remove_unused_track_caller

Conversation

@IWANABETHATGUY
Copy link
Member

@IWANABETHATGUY IWANABETHATGUY commented Jul 22, 2025

No description provided.

@IWANABETHATGUY IWANABETHATGUY mentioned this pull request Jul 22, 2025
Merged
@IWANABETHATGUY Graphite App
Copy link
Member Author

IWANABETHATGUY commented Jul 22, 2025 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@IWANABETHATGUY IWANABETHATGUY marked this pull request as ready for review July 22, 2025 12:30
@IWANABETHATGUY IWANABETHATGUY force-pushed the 07-22-chore_remove_unused_track_caller branch from 22d9e11 to fae9442 Compare July 22, 2025 12:42
@github-actions
Copy link
Contributor

github-actions bot commented Jul 22, 2025 •
edited
Loading

Benchmarks Rust

  • target: main(7076e9c)
  • pr: 07-22-chore_remove_unused_track_caller(6240dac)
group                                                        pr                                     target
-----                                                        --                                     ------
bundle/bundle@multi-duplicated-top-level-symbol              1.04     77.8±1.57ms        ? ?/sec    1.00     74.6±1.73ms        ? ?/sec
bundle/bundle@multi-duplicated-top-level-symbol-sourcemap    1.05     88.1±1.30ms        ? ?/sec    1.00     84.1±1.27ms        ? ?/sec
bundle/bundle@rome_ts                                        1.04    115.7±4.40ms        ? ?/sec    1.00    111.8±1.57ms        ? ?/sec
bundle/bundle@rome_ts-sourcemap                              1.02    136.8±2.21ms        ? ?/sec    1.00    133.8±1.50ms        ? ?/sec
bundle/bundle@threejs                                        1.04     41.7±0.90ms        ? ?/sec    1.00     40.3±0.99ms        ? ?/sec
bundle/bundle@threejs-sourcemap                              1.00     50.7±0.77ms        ? ?/sec    1.00     50.9±0.83ms        ? ?/sec
bundle/bundle@threejs10x                                     1.01    443.1±4.39ms        ? ?/sec    1.00    437.4±4.38ms        ? ?/sec
bundle/bundle@threejs10x-sourcemap                           1.00    524.4±3.34ms        ? ?/sec    1.00    522.2±6.22ms        ? ?/sec
scan/scan@rome_ts                                            1.00     87.1±1.31ms        ? ?/sec    1.03     90.2±1.15ms        ? ?/sec
scan/scan@threejs                                            1.00     30.4±1.93ms        ? ?/sec    1.00     30.3±0.32ms        ? ?/sec
scan/scan@threejs10x                                         1.00    319.9±4.69ms        ? ?/sec    1.02    325.9±4.27ms        ? ?/sec

Base automatically changed from 07-22-fix_5197 to main July 22, 2025 19:55
@IWANABETHATGUY IWANABETHATGUY force-pushed the 07-22-chore_remove_unused_track_caller branch from fae9442 to 6240dac Compare July 22, 2025 19:55
@netlify
Copy link

netlify bot commented Jul 22, 2025

✅ Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit 6240dac
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/687fecb06e6f8200081db733

@graphite-app
Copy link
Contributor

graphite-app bot commented Jul 22, 2025

Merge activity

  • Jul 22, 7:55 PM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.

@IWANABETHATGUY IWANABETHATGUY deleted the 07-22-chore_remove_unused_track_caller branch July 23, 2025 04:30
@IWANABETHATGUY
Copy link
Member Author

IWANABETHATGUY commented Jul 23, 2025

Seems not needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hyf0 hyf0 hyf0 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@IWANABETHATGUY @hyf0