Skip to content

fix(wasm): add wasm fallback in checkVersion#10097

Merged
h-a-n-a merged 1 commit intoweb-infra-dev:mainfrom
CPunisher:04-20-chore/wasm-version-check
Apr 21, 2025
Merged

fix(wasm): add wasm fallback in checkVersion#10097
h-a-n-a merged 1 commit intoweb-infra-dev:mainfrom
CPunisher:04-20-chore/wasm-version-check

Conversation

@CPunisher
Copy link
Contributor

Summary

This eliminate errors if users only install @rspack/binding-wasm32-wasi. Usually users are aware of they are choosing wasm binding so I think we can avoid introducing very detailed messages.

Checklist

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

@netlify
Copy link

netlify bot commented Apr 21, 2025

Deploy Preview for rspack ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 7e23909
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/6805b091f0253a0008f17bdc
😎 Deploy Preview https://deploy-preview-10097--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.

@CPunisher CPunisher requested a review from h-a-n-a April 21, 2025 02:42
@codspeed-hq
Copy link

codspeed-hq bot commented Apr 21, 2025

CodSpeed Performance Report

Merging #10097 will not alter performance

Comparing CPunisher:04-20-chore/wasm-version-check (7e23909) with main (9664a09)

Summary

✅ 11 untouched benchmarks

@h-a-n-a h-a-n-a changed the title chore(wasm): add wasm fallback in checkVersion fix(wasm): add wasm fallback in checkVersion Apr 21, 2025
@h-a-n-a h-a-n-a merged commit 8e75f78 into web-infra-dev:main Apr 21, 2025
31 checks passed
@h-a-n-a
Copy link
Contributor

h-a-n-a commented Apr 21, 2025

Thank you!

@github-actions github-actions bot added the release: bug fix release: bug related release(mr only) label Apr 21, 2025
@CPunisher CPunisher deleted the 04-20-chore/wasm-version-check branch April 21, 2025 09:30
@CPunisher CPunisher mentioned this pull request Apr 21, 2025
8 tasks
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