Skip to content

Update solana readmes#1916

Open
PabloMansanet wants to merge 6 commits intomainfrom
fix-readme
Open

Update solana readmes#1916
PabloMansanet wants to merge 6 commits intomainfrom
fix-readme

Conversation

@PabloMansanet
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions
Copy link
Copy Markdown

👋 PabloMansanet, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

Install Rust, Solana & Anchor. See https://solana.com/docs/intro/installation
- [Rust](https://www.rust-lang.org/tools/install) (version pinned via `rust-toolchain.toml`)
- [Solana CLI](https://docs.anza.xyz/cli/install) (provides `solana-test-validator`)
- [Anchor CLI](https://www.anchor-lang.com/docs/installation)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This repo requires anchor 0.29, which you may want to call out here. It also requires older solana & rust versions sadly

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Rust version I pinned in chains/solana/contracts/rust-toolchain.toml, which should force the right one to be installed + used. As for anchor, it's not quite 1:1 because this is about the Anchor CLI, which works all the way up to 0.32.1 (latest I think). I thought about restricting the CLI to be the same version as the anchor dependency, but I don't know if there's a reason to. Do you know if anything breaks when using a newer anchor CLI?

@github-actions
Copy link
Copy Markdown

Metric fix-readme main
Coverage 70.1% 69.8%

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.

3 participants