Repository navigation
feat: add support for new asset query suffixes - #21379
Conversation
🦋 Changeset detectedLatest commit: d8bb9d9 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #21379 +/- ##
=======================================
Coverage 92.92% 92.92%
=======================================
Files 601 601
Lines 68761 68763 +2
Branches 19350 19351 +1
=======================================
+ Hits 63898 63901 +3
+ Misses 4863 4862 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Merging this PR will not alter performance
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Memory | benchmark "asset-modules-resource", scenario '{"name":"mode-development-rebuild","mode":"development","watch":true}' |
175.4 KB | 1,475.6 KB | -88.12% |
| ⚡ | Memory | benchmark "devtool-eval", scenario '{"name":"mode-development-rebuild","mode":"development","watch":true}' |
333.3 KB | 142.5 KB | ×2.3 |
| ⚡ | Memory | benchmark "asset-modules-inline", scenario '{"name":"mode-development-rebuild","mode":"development","watch":true}' |
322 KB | 215.5 KB | +49.45% |
| ⚡ | Memory | benchmark "many-modules-esm", scenario '{"name":"mode-development-rebuild","mode":"development","watch":true}' |
298.4 KB | 218.9 KB | +36.32% |
| ⚡ | Memory | benchmark "many-chunks-commonjs", scenario '{"name":"mode-production","mode":"production"}' |
10.6 MB | 7.9 MB | +33.04% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing bjohansebas:assets (d8bb9d9) with main (5798eda)
…ne`, and `?no-inline`
|
This PR is packaged and the instant preview is available (0f1df89). Install it locally:
npm i -D webpack@https://pkg.pr.new/webpack@0f1df89
yarn add -D webpack@https://pkg.pr.new/webpack@0f1df89
pnpm add -D webpack@https://pkg.pr.new/webpack@0f1df89 |
…ne
, and?no-inline`Summary
closes #333.
What kind of change does this PR introduce?
Did you add tests for your changes?
Does this PR introduce a breaking change?
If relevant, what needs to be documented once your changes are merged or what have you already documented?
Use of AI