Skip to content

fix: derive rspack.target from libConfig.syntax#213

Merged
Timeless0911 merged 1 commit intomainfrom
libconfig-syntax
Sep 18, 2024
Merged

fix: derive rspack.target from libConfig.syntax#213
Timeless0911 merged 1 commit intomainfrom
libconfig-syntax

Conversation

@fi3ework
Copy link
Copy Markdown
Member

@fi3ework fi3ework commented Sep 18, 2024

Summary

Fix #209.

  • only webpack runtime related code will be affected.
  • need to manually assign {type}loading as it derive the value from target by default, which will lead to unimpelmented universal type.

Related Links

Checklist

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

@netlify
Copy link
Copy Markdown

netlify bot commented Sep 18, 2024

Deploy Preview for rslib ready!

Name Link
🔨 Latest commit
🔍 Latest deploy log https://app.netlify.com/sites/rslib/deploys/66ea8376b2c8a1299043f378
😎 Deploy Preview https://deploy-preview-213--rslib.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
Copy link
Copy Markdown
Member Author

@Timeless0911
Copy link
Copy Markdown
Contributor

main/netlify.toml is not working 🤨.

I rerun it, it seems work. Try to force push again to verify.

@fi3ework
Copy link
Copy Markdown
Member Author

fi3ework commented Sep 18, 2024

@Timeless0911 https://app.netlify.com/sites/rslib/deploys

Google Chrome 2024-09-18 15 55 56

Weird, the below one was triggered by this PR, which is canceled by as expected. The above one still finish the preview build and deploy. We could test in following PRs, PLAT this PR.

@Timeless0911 Timeless0911 merged commit 057cd87 into main Sep 18, 2024
@Timeless0911 Timeless0911 deleted the libconfig-syntax branch September 18, 2024 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: should not set target: 'es5' when using syntax

2 participants