Skip to content

ci(contract): add MSRV matrix job without Stellar CLI#803

Merged
aji70 merged 1 commit intoMyFanss:mainfrom
levoski1:ci/contract-msrv-matrix
Apr 24, 2026
Merged

ci(contract): add MSRV matrix job without Stellar CLI#803
aji70 merged 1 commit intoMyFanss:mainfrom
levoski1:ci/contract-msrv-matrix

Conversation

@levoski1
Copy link
Copy Markdown
Contributor

  • Add rust-version = "1.74" to contract/Cargo.toml [workspace.package] (soroban-sdk 21.x lower bound driven by curve25519-dalek 4.x)
  • Add .github/workflows/contract-msrv.yml: optional second CI job that installs the declared MSRV toolchain via dtolnay/rust-toolchain, runs cargo check --locked and cargo test --locked --all-features
  • No Stellar CLI installed; soroban-sdk testutils are pure-Rust
  • Separate path filter (contract/**) keeps it from running on unrelated changes
  • Matrix key 'rust' makes it trivial to add more versions later
  • Cargo registry and target caches are keyed per rust version to avoid cross-toolchain cache pollution

closes #611

- Add rust-version = "1.74" to contract/Cargo.toml [workspace.package]
  (soroban-sdk 21.x lower bound driven by curve25519-dalek 4.x)
- Add .github/workflows/contract-msrv.yml: optional second CI job that
  installs the declared MSRV toolchain via dtolnay/rust-toolchain, runs
  cargo check --locked and cargo test --locked --all-features
- No Stellar CLI installed; soroban-sdk testutils are pure-Rust
- Separate path filter (contract/**) keeps it from running on unrelated changes
- Matrix key 'rust' makes it trivial to add more versions later
- Cargo registry and target caches are keyed per rust version to avoid
  cross-toolchain cache pollution
@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented Apr 23, 2026

@levoski1 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@aji70 aji70 merged commit 0860910 into MyFanss:main Apr 24, 2026
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.

Workspace MSRV matrix without Stellar CLI

2 participants