Skip to content

fix: webpackexports comment#11597

Merged
LingyuCoder merged 10 commits intoweb-infra-dev:mainfrom
zhangyuang:fix/magic-comment-webpackexports
Sep 10, 2025
Merged

fix: webpackexports comment#11597
LingyuCoder merged 10 commits intoweb-infra-dev:mainfrom
zhangyuang:fix/magic-comment-webpackexports

Conversation

@zhangyuang
Copy link
Contributor

Summary

If magic comments with webpackExports, the iterator will be finished, which causes the comments to remain and can't be collected.

Related links

#11583

Checklist

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

@netlify
Copy link

netlify bot commented Sep 5, 2025

Deploy Preview for rspack ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit ffab63b
🔍 Latest deploy log https://app.netlify.com/projects/rspack/deploys/68c1076931103e0008ba2d35
😎 Deploy Preview https://deploy-preview-11597--rspack.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@CLAassistant
Copy link

CLAassistant commented Sep 5, 2025

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added the release: bug fix release: bug related release(mr only) label Sep 5, 2025
@zhangyuang zhangyuang changed the title fix: webpack magic comment fix: webpackexports comment Sep 5, 2025
Copy link
Contributor

@LingyuCoder LingyuCoder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, the test cases should be added in tests/rspack-test. The test/webpack-test is using to migrate test cases from webpack and should not be modified.

@zhangyuang
Copy link
Contributor Author

Okay, let me update it.

@codspeed-hq
Copy link

codspeed-hq bot commented Sep 10, 2025

CodSpeed Performance Report

Merging #11597 will not alter performance

Comparing zhangyuang:fix/magic-comment-webpackexports (ffab63b) with main (3f6229a)

Summary

✅ 17 untouched benchmarks

@zhangyuang
Copy link
Contributor Author

Has updated

Copy link
Contributor

@LingyuCoder LingyuCoder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks

@LingyuCoder LingyuCoder merged commit 9dd9d3d into web-infra-dev:main Sep 10, 2025
44 checks passed
@i5ting
Copy link

i5ting commented Sep 11, 2025

棒棒哒

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)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants