Skip to content

chore: convert rust-toolchain to TOML and add wasm32-wasip1 target#54

Merged
vagetman merged 2 commits into
mainfrom
vadim/rust-toolchain-toml
Jun 12, 2026
Merged

chore: convert rust-toolchain to TOML and add wasm32-wasip1 target#54
vagetman merged 2 commits into
mainfrom
vadim/rust-toolchain-toml

Conversation

@vagetman

@vagetman vagetman commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

The release workflow failed with:

error[E0463]: can't find crate for `std`
  = note: the `wasm32-wasip1` target may not be installed

The reusable workflow (fastly/devex-reusable-workflows) uses actions-rust-lang/setup-rust-toolchain which reads rust-toolchain.toml and installs declared targets automatically, but the old plain-text rust-toolchain file only specified the channel version.

Changes

  • Renamed rust-toolchain -> rust-toolchain.toml
  • Added targets = ["wasm32-wasip1"] so the WASI target is installed automatically in CI

No functional changes — same Rust 1.95.0 toolchain.

vagetman added 2 commits June 9, 2026 10:33
The release workflow failed because the wasm32-wasip1 target was not
installed. The reusable workflow uses actions-rust-lang/setup-rust-toolchain
which reads rust-toolchain.toml and installs declared targets automatically.

Convert the plain-text rust-toolchain file to rust-toolchain.toml with the
targets field so CI installs wasm32-wasip1 without manual steps.
@vagetman

vagetman commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator Author

TLDR; actions/checkout@v2 couldn't read rust-toolchain.toml only rust-toolchain. replaced to actions/checkout@v4 that can handle TOML correctly

@vagetman vagetman requested a review from kailan June 9, 2026 16:22
@vagetman vagetman merged commit a92922b into main Jun 12, 2026
3 checks passed
@vagetman vagetman deleted the vadim/rust-toolchain-toml branch June 12, 2026 16:10
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.

2 participants