Skip to content

fix: use evaluate expression to extract hmr accept dependencies#10107

Merged
ahabhgk merged 3 commits intomainfrom
more-wp-hmr-tests
Apr 22, 2025
Merged

fix: use evaluate expression to extract hmr accept dependencies#10107
ahabhgk merged 3 commits intomainfrom
more-wp-hmr-tests

Conversation

@ahabhgk
Copy link
Copy Markdown
Contributor

@ahabhgk ahabhgk commented Apr 21, 2025

Summary

fix and enable more webpack hmr tests

the remaining not passed test cases:

not bug related to hmr:

  • css/fetch-priority: not support fetch priority for css
  • loader-import-module/css: bug on importModule
  • lazy-compilation/*: we use middleware, all tests are migrated in rspack-test-tools

bug related to hmr but probably nobody will encounter:

  • hashing/full-hash-syntax-error: bug on emitOnErrors
  • runtime/root-error: bug on emitOnErrors

Checklist

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

@github-actions github-actions bot added the team The issue/pr is created by the member of Rspack. label Apr 21, 2025
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 21, 2025

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit 74fb35c
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/680633ae558d5e0008ce870d

@github-actions github-actions bot added the release: bug fix release: bug related release(mr only) label Apr 21, 2025
@ahabhgk ahabhgk requested a review from LingyuCoder April 21, 2025 09:42
@ahabhgk ahabhgk enabled auto-merge (squash) April 21, 2025 09:46
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Apr 21, 2025

CodSpeed Performance Report

Merging #10107 will not alter performance

Comparing more-wp-hmr-tests (74fb35c) with main (ad4ba18)

Summary

✅ 11 untouched benchmarks

@ahabhgk ahabhgk merged commit 3edc2c1 into main Apr 22, 2025
29 checks passed
@ahabhgk ahabhgk deleted the more-wp-hmr-tests branch April 22, 2025 02:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: bug fix release: bug related release(mr only) team The issue/pr is created by the member of Rspack.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants