-
Notifications
You must be signed in to change notification settings - Fork 698
Closed
Description
As https://rollupjs.org/configuration-options/#context shown that the default value of options.context is "undefined", however, in Rolldown, the default value of options.context is currently an empty string (""), and it uses void 0 actually to replace the top-level context, leading to a discrepancy in default values ("void 0" vs."undefined" vs. "").
| context: raw_options.context.unwrap_or_default(), |
rolldown/crates/rolldown_common/src/inner_bundler_options/types/normalized_bundler_options.rs
Line 177 in beec4a4
| context: Default::default(), |
Should we align Rolldown with Rollup by setting the default value of options.context to "void 0" (in Rollup, default value is "undefined") to ensure consistency?
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels