-
-
Notifications
You must be signed in to change notification settings - Fork 6.1k
Replace webpack with rspack #35460
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace webpack with rspack #35460
Conversation
One related update added here: https://github.com/sindresorhus/add-asset-webpack-plugin/releases/tag/v3.1.0 includes type definitions, so I removed the type stub for that package. |
This is working very well in my testing, chunk loading and all works as expected. It supports the webpack comments and variables, so from my point of view, this is ready. |
It seems rspack haven't support riscv yet.
|
Yep, and it seems we are the first to hit this, I opened web-infra-dev/rspack#11656. We should also ensure our PR pipeline does this RISC build. @techknowlogick do we have any download stats on our docker image on how many RISC-V Users there actually are? I wonder if it's really worth to support this platform. |
* giteaofficial/main: Fix different behavior in status check pattern matching with double stars (go-gitea#35474) Replace webpack with rspack (go-gitea#35460) Don't store repo archives on `gitea dump` (go-gitea#35467) Fix SSH signing key path will be displayed in the pull request UI (go-gitea#35381) [skip ci] Updated translations via Crowdin Update image name in integration README (go-gitea#35465)
I guess we could make it use webpack on that specific platform. But tbh I'm tired of RISC-V issues, it's just not well-supported by anything. |
I'd rather go back to 2-part version syntax in go.mod. OP's issue sounds like a go bug. |
Maybe we can revert the change at the moment and wait that issue fixed from rspack. We just encountered 2 issues of RISC-V. |
Sure, revert it if its blocking the release. I do expect rspack to eventually fix it. |
This reverts commit 866c636.
Partially revert #35460 until rspack supports RISC-V (web-infra-dev/rspack#11656).
* origin/main: Clean up npm dependencies (go-gitea#35484) Update eslint to v9 (go-gitea#35485) Revert the rspack change (go-gitea#35482) Replace gobwas/glob package (go-gitea#35478) Fix various typos in codebase (go-gitea#35480) Fix different behavior in status check pattern matching with double stars (go-gitea#35474) Replace webpack with rspack (go-gitea#35460) Don't store repo archives on `gitea dump` (go-gitea#35467)
Given that this bundler is almost a drop-in replacement to webpack, it might be worth switching. So far it seems everything is working, but more testing is needed, so I'm setting draft.