Skip to content

docs(scripts): document Solana CLI prerequisite - #204

Merged
jackchuma merged 1 commit into
mainfrom
docs/chain-4912-solana-cli-prerequisite
Oct 13, 2026
Merged

jackchuma merged 1 commit into
mainfrom
docs/chain-4912-solana-cli-prerequisite

Conversation

@jackchuma

Copy link
Copy Markdown
Collaborator

Linear: https://linear.app/coinbase/issue/CHAIN-4912/improve-solana-bridge-quick-start-guide

What changed? Why?

The scripts quick start uses solana-keygen new without explaining how to install the required CLI. Add a link to the official Solana CLI installation instructions before that command and tell readers to install the CLI first.

Notes to reviewers

Only scripts/README.md changes. Existing commands remain unchanged. CLI installation, key generation, and bridge operations are out of scope.

How has it been tested?

  • git diff --check origin/main...HEAD passes.
  • The installation page returns HTTP 200. Verified its install-solana-cli anchor, Agave installer instructions, and solana --version verification command.
  • No tests added for this documentation-only change. No installation, key generation, signing, or transactions were run.

Generated with Toshi

@cb-heimdall

cb-heimdall commented Oct 13, 2026 •

Copy link
Copy Markdown
Collaborator

✅ Heimdall Review Status

Requirement Status More Info
Reviews ✅ 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

⚠️ Ignored Reviews (4)

Reviewer Reason
xenoliss Old or outdated review
0xth4nh Soft review, as user does not have write permissions
xenoliss Soft review, as user does not have write permissions
xenoliss Review missing MFA

@linear

linear Bot commented Oct 13, 2026

Copy link
Copy Markdown

CHAIN-4912

@jackchuma
jackchuma marked this pull request as ready for review October 13, 2026 13:04
@jackchuma
jackchuma requested review from 0xth4nh and xenoliss October 13, 2026 13:24
@0xth4nh

0xth4nh commented Oct 13, 2026

Copy link
Copy Markdown
Collaborator

LGTM !!!

@jackchuma
jackchuma merged commit 2bbfe1f into main Oct 13, 2026
7 checks passed
@jackchuma
jackchuma deleted the docs/chain-4912-solana-cli-prerequisite branch October 13, 2026 18:01
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.

4 participants