-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
comprehensive-demo-react18: inline bootstrap + async startup #4406
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
… (webpack/rspack)
| experiments: { | ||
| css: true, | ||
| mfAsyncStartup: true, | ||
| }, |
Check failure
Code scanning / CodeQL
Overwritten property Error
another property
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 2 days ago
To fix this problem, combine the contents of both experiments properties into a single definition, ensuring that no desired fields are lost. Only one experiments property should remain. The fix should be performed in comprehensive-demo-react18/app-02/rspack.config.js, specifically from lines 21–24 and 64–66. Merge the properties so that experiments contains both css: true and mfAsyncStartup: true. Remove the duplicate second definition (experiments at lines 64–66), and ensure only one experiments block remains in the exported configuration object.
No imports or additional definitions are required.
-
Copy modified line R64
| @@ -61,9 +61,7 @@ | ||
| }, | ||
| ], | ||
| }, | ||
| experiments: { | ||
| css: true, | ||
| }, | ||
| // Removed duplicate 'experiments' property. See line 21 for the correct, merged definition. | ||
|
|
||
| plugins: [ | ||
| new ModuleFederationPlugin({ |
… for canary version 1.6.2-canary-4e6be088-20251110224718
2dffd40 to
962a7ec
Compare
…ackages to canary versions
…rver/plugin packages
…package names (@rspack-canary/core, etc.)
Remove dynamic entry imports, inline bootstrap in index, enable async startup across webpack (experiments.asyncStartup) and Rspack (experiments.mfAsyncStartup).