Skip to content

fix(ci): align wasm target to wasm32v1-none across deploy, docs, and CI - #17

Merged
cybermax4200 merged 1 commit into
ecotask-network:mainfrom
devprom6:fix/ci-wasm-target-alignment
Jul 13, 2026
Merged

fix(ci): align wasm target to wasm32v1-none across deploy, docs, and CI#17
cybermax4200 merged 1 commit into
ecotask-network:mainfrom
devprom6:fix/ci-wasm-target-alignment

Conversation

@devprom6

Copy link
Copy Markdown
Contributor

Closes #6

Problem

CI and Makefile built with wasm32v1-none, but deploy.sh, README.md, and CONTRIBUTING.md referenced the legacy wasm32-unknown-unknown target. This meant CI artifacts couldn't be used by deploy scripts, and local builds following the README would use a different target than CI.

Changes

  • deploy.sh: Updated WASM output path and error message to wasm32v1-none
  • README.md: Updated all 5 references (rustup target add, build commands, deploy example)
  • CONTRIBUTING.md: Updated 2 references + added note explaining the difference between wasm32v1-none (Soroban SDK v20+) and wasm32-unknown-unknown (legacy)

Target choice rationale

wasm32v1-none is the recommended target for Soroban SDK v20+. It produces smaller WASM binaries with lower deployment costs. CI and the Makefile were already correct; docs and deploy scripts needed alignment.

Fixes ecotask-network#6. Deploy scripts and README/CONTRIBUTING docs referenced the
legacy wasm32-unknown-unknown target while CI and Makefile already used
wasm32v1-none. Aligns all files to wasm32v1-none and adds a contributor
note explaining the difference between the two targets.
@cybermax4200
cybermax4200 merged commit 0964e6d into ecotask-network:main Jul 13, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: Pinned wasm32v1-none target doesn't match deploy scripts

2 participants