-
Notifications
You must be signed in to change notification settings - Fork 353
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
chore(builder): simplify assets rule #4701
Conversation
🦋 Changeset detectedLatest commit: f07bbe2 The changes in this PR will be included in the next version bump. This PR includes changesets to release 257 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #4701 +/- ##
==========================================
- Coverage 57.59% 53.73% -3.86%
==========================================
Files 672 664 -8
Lines 17794 18108 +314
Branches 3876 4147 +271
==========================================
- Hits 10249 9731 -518
- Misses 6934 7718 +784
- Partials 611 659 +48 ☔ View full report in Codecov by Sentry. |
Summary
🤖 Generated by Copilot at 6466f7e
This pull request updates the
@modern-js/builder
package to use the new asset module system in webpack 5. It changes the webpack rules for static assets and SVG files in theconfig.ts
andsvg.ts
files, and adds a.changeset
file to document the patch bump.Details
🤖 Generated by Copilot at 6466f7e
@modern-js/builder
package (link)type: 'asset/resource'
and remove unnecessary options (link, link)type: 'asset/inline'
andtype: 'asset/resource'
withgenerator
option (link)config.ts
andsvg.ts
files inpackages/builder/builder-shared
andpackages/builder/builder
folders respectively (link, link)Related Issue
Checklist
pnpm run change
.