Skip to content

Conversation

fi3ework
Copy link
Member

@fi3ework fi3ework commented Aug 15, 2024

Summary

update: port webpack/webpack#18696.


Port webpack/webpack#18660, please take a look at the description from the webpack PR. This is my first time editing binding related code, hope didn't break any implicit things.

Checklist

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

@github-actions github-actions bot added the release: bug fix release: bug related release(mr only) label Aug 15, 2024
Copy link

netlify bot commented Aug 15, 2024

Deploy Preview for rspack ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit ce2912c
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/66c7fa4b5e25460008a40df7
😎 Deploy Preview https://deploy-preview-7585--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 site configuration.

@fi3ework fi3ework force-pushed the external-fallback branch 4 times, most recently from 1045e84 to 892480e Compare August 15, 2024 09:44
@fi3ework fi3ework requested a review from ahabhgk August 15, 2024 09:47
@fi3ework fi3ework marked this pull request as ready for review August 15, 2024 10:53
@fi3ework fi3ework force-pushed the external-fallback branch 2 times, most recently from 84d893d to 74bbb51 Compare August 16, 2024 10:32
@fi3ework fi3ework changed the title fix: module-import get fallback from externalsPresets fix: "module-import" simply falls back to "module" when it fails. Aug 23, 2024
@fi3ework fi3ework changed the title fix: "module-import" simply falls back to "module" when it fails. fix: "module-import" simply falls back to "module" when bails Aug 23, 2024
@fi3ework fi3ework requested a review from ahabhgk August 23, 2024 04:40
@fi3ework fi3ework merged commit 90a56d0 into web-infra-dev:main Aug 23, 2024
@fi3ework fi3ework deleted the external-fallback branch August 23, 2024 05:29
@chenjiahan chenjiahan mentioned this pull request Aug 23, 2024
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.

2 participants