Skip to content

feat: dataUrlCondition support function#13053

Open
lilei946 wants to merge 1 commit intoweb-infra-dev:mainfrom
lilei946:main
Open

feat: dataUrlCondition support function#13053
lilei946 wants to merge 1 commit intoweb-infra-dev:mainfrom
lilei946:main

Conversation

@lilei946
Copy link

Summary

The pr is to support function for dataUrlCondition, which is to determined the assets should be inlined or emitted into the output directory.

Related links

webpack: https://webpack.js.org/configuration/module/#ruleparserdataurlcondition

issue: #8488

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@lilei946 lilei946 requested a review from hardfist as a code owner February 12, 2026 12:32
@github-actions github-actions bot added the release: feature release: feature related release(mr only) label Feb 12, 2026
@codspeed-hq
Copy link

codspeed-hq bot commented Feb 13, 2026

Merging this PR will improve performance by 82.09%

⚡ 2 improved benchmarks
✅ 15 untouched benchmarks
⏩ 1 skipped benchmark1

Performance Changes

Benchmark BASE HEAD Efficiency
bundle@threejs-development 820.4 ms 724.2 ms +13.28%
rust@build_module_graph 2.6 s 1.4 s +82.09%

Comparing lilei946:main (ea0a6b5) with main (dcb192a)2

Open in CodSpeed

Footnotes

  1. 1 benchmark was skipped, so the baseline result was used instead. If it was deleted from the codebase, click here and archive it to remove it from the performance reports.

  2. No successful run was found on main (090d7ba) during the generation of this report, so dcb192a was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: feature release: feature related release(mr only)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant