Skip to content

Comments

feat(debug): only emit debug information for trace level#5108

Merged
hyf0 merged 1 commit intomainfrom
06-30-feat_debug_only_emit_debug_information_for_trace_level
Jun 30, 2025
Merged

feat(debug): only emit debug information for trace level#5108
hyf0 merged 1 commit intomainfrom
06-30-feat_debug_only_emit_debug_information_for_trace_level

Conversation

@hyf0
Copy link
Member

@hyf0 hyf0 commented Jun 30, 2025

No description provided.

@hyf0 hyf0 marked this pull request as ready for review June 30, 2025 07:55
Copy link
Member Author

hyf0 commented Jun 30, 2025

Base automatically changed from 06-30-feat_debug_emit_chunk_related_informations to main June 30, 2025 07:57
@hyf0 hyf0 force-pushed the 06-30-feat_debug_only_emit_debug_information_for_trace_level branch from feef00f to f610124 Compare June 30, 2025 09:12
@netlify
Copy link

netlify bot commented Jun 30, 2025

Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit f610124
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/686254fa02d414000877a3bd

@github-actions
Copy link
Contributor

Benchmarks Rust

  • target: main(67b295f)
  • pr: 06-30-feat_debug_only_emit_debug_information_for_trace_level(f610124)
group                                                        pr                                     target
-----                                                        --                                     ------
bundle/bundle@multi-duplicated-top-level-symbol              1.00     76.6±3.88ms        ? ?/sec    1.02     78.5±2.37ms        ? ?/sec
bundle/bundle@multi-duplicated-top-level-symbol-sourcemap    1.03     89.8±2.96ms        ? ?/sec    1.00     86.8±1.27ms        ? ?/sec
bundle/bundle@rome_ts                                        1.00    116.9±2.48ms        ? ?/sec    1.00    116.6±4.07ms        ? ?/sec
bundle/bundle@rome_ts-sourcemap                              1.01    135.7±2.91ms        ? ?/sec    1.00    134.1±1.83ms        ? ?/sec
bundle/bundle@threejs                                        1.00     42.0±2.39ms        ? ?/sec    1.01     42.4±2.62ms        ? ?/sec
bundle/bundle@threejs-sourcemap                              1.03     51.9±2.70ms        ? ?/sec    1.00     50.4±0.96ms        ? ?/sec
bundle/bundle@threejs10x                                     1.03    447.7±7.77ms        ? ?/sec    1.00    435.7±6.37ms        ? ?/sec
bundle/bundle@threejs10x-sourcemap                           1.01    515.7±6.57ms        ? ?/sec    1.00    511.9±6.78ms        ? ?/sec
scan/scan@rome_ts                                            1.03     90.8±1.96ms        ? ?/sec    1.00     88.0±1.86ms        ? ?/sec
scan/scan@threejs                                            1.01     31.9±0.74ms        ? ?/sec    1.00     31.6±1.83ms        ? ?/sec
scan/scan@threejs10x                                         1.02    327.7±4.41ms        ? ?/sec    1.00    321.4±4.54ms        ? ?/sec

@hyf0 hyf0 requested a review from shulaoda June 30, 2025 09:30
@hyf0 hyf0 added this pull request to the merge queue Jun 30, 2025
Merged via the queue into main with commit 9ccb35b Jun 30, 2025
22 checks passed
@hyf0 hyf0 deleted the 06-30-feat_debug_only_emit_debug_information_for_trace_level branch June 30, 2025 09:38
This was referenced Jul 1, 2025
hyf0 pushed a commit that referenced this pull request Jul 1, 2025
## [1.0.0-beta.23] - 2025-07-01

### 💥 BREAKING CHANGES

- debug: rename debug logs fields to be consitent (#5125) by @antfu

### 🚀 Features

- debug: append `content` for asset data and unify `source` field to
`content`. (#5130) by @hyf0
- debug: emit asset-related data (#5124) by @hyf0
- commonjs tree shaking. (#5095) by @IWANABETHATGUY
- rolldown_plugin_wasm_helper: align behaviors for expected
functionality (#5120) by @shulaoda
- remove `experimental.enableComposingJsPlugins` (#5122) by @shulaoda
- use same option object reference (#5114) by @sapphi-red
- add util function to inspect why the facade symbol is created (#5115)
by @IWANABETHATGUY
- types: expose `ChunkingContext` type (#5112) by @sapphi-red
- scan commonjs exports (#5111) by @IWANABETHATGUY
- debug: remove unneeded source info of render chunk events (#5109) by
@hyf0
- debug: only emit debug information for trace level (#5108) by @hyf0
- debug: emit chunk related informations (#5106) by @hyf0

### 🐛 Bug Fixes

- debug: add hook render chunk start and end events to Meta enum (#5117)
by @hyf0

### 🚜 Refactor

- rust: unify outdated namings (#5127) by @hyf0

### 📚 Documentation

- advanced-chunks: clarify module capturing behavior (#5129) by @hyf0

### 🧪 Testing

- support array output options (#5113) by @sapphi-red

### ⚙️ Miscellaneous Tasks

- remove unused code (#5128) by @IWANABETHATGUY
- remove unnecessary `skipComposingJsPlugin` (#5123) by @shulaoda
- deps: update crate-ci/typos action to v1.34.0 (#5119) by
@renovate[bot]
- deps: update NAPI-RS to 3.0.0-beta.11 (#5110) by @shulaoda

Co-authored-by: shulaoda <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants