Skip to content
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

Amend PRR #2474

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Amend PRR #2474

wants to merge 1 commit into from

Conversation

AurevoirXavier
Copy link
Contributor

@AurevoirXavier AurevoirXavier commented Dec 22, 2024

After conducting some research, I realized the node tools is currently ineffective. More and more teams decide to release their runtimes separately. For example, Polkadot, Acala, Moonbeam and..

But I also worked on other tasks. Initially, I thought it would be beneficial to use subwasm and subwasmlib, but I discovered that the crate is overly complex and bloated with unnecessary dependencies. Additionally, its version management is poorly maintained (e.g., this issue has persisted unresolved for over a year).

As a result, I decided to develop my own version, which provides a streamlined subset of subwasm‘s functionality. The goal isn’t to replace subwasm entirely but to enhance the release process. Specifically, it aims to replace subwasm within the release pipeline. My implementation natively interacts with Docker in Rust, offers improved error handling compared to shell-based approaches, and inspects essential information from WASM files. This makes it a much more practical tool for the release pipeline.

Consequently, I updated the deliverables by replacing the node tools with this new solution.

@github-actions github-actions bot added the admin-review This application requires a review from an admin. label Dec 22, 2024
@AurevoirXavier AurevoirXavier force-pushed the master branch 3 times, most recently from 6e66b6a to 5a132ce Compare December 22, 2024 14:00
Signed-off-by: Xavier Lau <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
admin-review This application requires a review from an admin.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant