Skip to content

Comments

docs: clarify that advancedChunks options are in bytes#5022

Merged
hyf0 merged 1 commit intomainfrom
docs/clarify-advanced-chunks-options-are-in-bytes
Jun 20, 2025
Merged

docs: clarify that advancedChunks options are in bytes#5022
hyf0 merged 1 commit intomainfrom
docs/clarify-advanced-chunks-options-are-in-bytes

Conversation

@sapphi-red
Copy link
Member

I guess vitejs/rolldown-vite#250 thought the values are in kilobytes. But I guess these are in bytes.

@sapphi-red sapphi-red requested a review from hyf0 June 20, 2025 01:38
@netlify
Copy link

netlify bot commented Jun 20, 2025

Deploy Preview for rolldown-rs ready!

Name Link
🔨 Latest commit 0877e3c
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/6854bbacb0aeee0008232e04
😎 Deploy Preview https://deploy-preview-5022--rolldown-rs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@hyf0
Copy link
Member

hyf0 commented Jun 20, 2025

Thank you!

@hyf0 hyf0 added this pull request to the merge queue Jun 20, 2025
Merged via the queue into main with commit c4bb779 Jun 20, 2025
21 checks passed
@hyf0 hyf0 deleted the docs/clarify-advanced-chunks-options-are-in-bytes branch June 20, 2025 05:00
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