Skip to content

Comments

refactor(rolldown_utils): simplify block_on_spawn_all#5019

Merged
shulaoda merged 1 commit intomainfrom
06-19-refactor_rolldown_utils_simplify_block_on_spawn_all
Jun 19, 2025
Merged

refactor(rolldown_utils): simplify block_on_spawn_all#5019
shulaoda merged 1 commit intomainfrom
06-19-refactor_rolldown_utils_simplify_block_on_spawn_all

Conversation

@shulaoda
Copy link
Member

@shulaoda shulaoda commented Jun 19, 2025

Simplify the function declarations of block_on_spawn_all under different cfg conditions.

Copy link
Member Author

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

@netlify
Copy link

netlify bot commented Jun 19, 2025

Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit 756e03d
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/68541e8a2b6c7d00081352cf

@shulaoda shulaoda changed the title refactor(rolldown_utils): simplify block_on_spawn_all refactor(rolldown_utils): simplify block_on_spawn_all Jun 19, 2025
@shulaoda shulaoda marked this pull request as ready for review June 19, 2025 14:29
@shulaoda shulaoda requested a review from hyf0 June 19, 2025 14:29
@github-actions
Copy link
Contributor

Benchmarks Rust

  • target: main(f5b8ec0)
  • pr: 06-19-refactor_rolldown_utils_simplify_block_on_spawn_all(756e03d)
group                                                        pr                                     target
-----                                                        --                                     ------
bundle/bundle@multi-duplicated-top-level-symbol              1.00     77.4±1.80ms        ? ?/sec    1.02     78.8±2.28ms        ? ?/sec
bundle/bundle@multi-duplicated-top-level-symbol-sourcemap    1.00     87.5±1.82ms        ? ?/sec    1.03     90.5±2.96ms        ? ?/sec
bundle/bundle@rome_ts                                        1.00    115.0±2.22ms        ? ?/sec    1.00    114.9±1.65ms        ? ?/sec
bundle/bundle@rome_ts-sourcemap                              1.00    129.8±2.04ms        ? ?/sec    1.00    129.6±1.92ms        ? ?/sec
bundle/bundle@threejs                                        1.01     42.1±1.71ms        ? ?/sec    1.00     41.6±2.19ms        ? ?/sec
bundle/bundle@threejs-sourcemap                              1.02     48.9±1.00ms        ? ?/sec    1.00     48.0±0.93ms        ? ?/sec
bundle/bundle@threejs10x                                     1.00    437.9±5.02ms        ? ?/sec    1.01    442.9±6.04ms        ? ?/sec
bundle/bundle@threejs10x-sourcemap                           1.00    511.5±6.04ms        ? ?/sec    1.01    517.0±5.79ms        ? ?/sec
scan/scan@rome_ts                                            1.03     91.0±2.96ms        ? ?/sec    1.00     88.6±1.65ms        ? ?/sec
scan/scan@threejs                                            1.00     31.4±0.85ms        ? ?/sec    1.00     31.5±1.93ms        ? ?/sec
scan/scan@threejs10x                                         1.01    327.7±5.38ms        ? ?/sec    1.00    323.5±4.45ms        ? ?/sec

@shulaoda shulaoda added this pull request to the merge queue Jun 19, 2025
Merged via the queue into main with commit 44c6f6d Jun 19, 2025
27 checks passed
@shulaoda shulaoda deleted the 06-19-refactor_rolldown_utils_simplify_block_on_spawn_all branch June 19, 2025 16:56
hyf0 pushed a commit that referenced this pull request Jun 22, 2025
## [1.0.0-beta.19] - 2025-06-22

### 🚀 Features

- support `OutputOptions#manualChunks` (#5037) by @hyf0
- advanced-chunks: support `advancedChunks#gruop#name` to be function
(#5035) by @hyf0
- rolldown_plugin_import_glob: align with `vitejs/vite#20163` (#5034) by
@shulaoda
- rust/advanced-chunks: support `MatchGroup#name` to be dynamic (#5033)
by @hyf0
- rolldown_plugin_build_import_analysis: align with `vitejs/vite#20117`
(#5027) by @shulaoda
- rolldown_plugin_build_import_analysis: align with `vitejs/vite#20115`
(#5020) by @shulaoda
- add validation warning for advanced chunks options without groups
(#5009) by @sapphi-red

### 🐛 Bug Fixes

- moduleInfo is not updated when entry module is emitted by
this.emitFile (#5032) by @IWANABETHATGUY
- preserveEntrySignatures: false generates circular imports that hangs
with TLA (#5029) by @IWANABETHATGUY
- rolldown_plugin_build_import_analysis: align pure dynamic import
handling with rolldown-vite (#5016) by @shulaoda
- plugin/vite-resolve: normalize leading slash (#5013) by @sapphi-red
- debug: `build_id` doesn't increase (#5015) by @hyf0
- side effects in this.emitFile({ type: 'chunk' }) is removed when
preserveEntrySignatures: false is set (#5012) by @IWANABETHATGUY

### 🚜 Refactor

- rolldown_utils: simplify `block_on_spawn_all` (#5019) by @shulaoda
- use `rolldown_utils::futures::block_on` for `WatcherImpl#start`
(#5018) by @shulaoda

### 📚 Documentation

- jsdoc: document `experimental.attachDebugInfo` (#5028) by @hyf0
- clarify that `advancedChunks` options are in bytes (#5022) by
@sapphi-red
- add a note that sequential conversion may break the code (#5024) by
@sapphi-red

### ⚙️ Miscellaneous Tasks

- infra: clean up `dist` before building `rolldown` (#5036) by @hyf0
- Align status notice in readme with documentation (#5021) by
@rijkvanzanten

### ❤️ New Contributors

* @rijkvanzanten made their first contribution in
[#5021](#5021)

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