Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Install Solana before using it #8638

Merged
merged 2 commits into from
Mar 4, 2020

Conversation

garious
Copy link
Contributor

@garious garious commented Mar 4, 2020

Problem

Docs start using the Solana CLI before the instructions on how to install it.

Summary of Changes

Move installation instructions to the top.

Fixes #8627

@garious garious requested a review from t-nelson March 4, 2020 22:29
@garious garious added the v1.0 label Mar 4, 2020
t-nelson
t-nelson previously approved these changes Mar 4, 2020
Copy link
Contributor

@t-nelson t-nelson left a comment

Choose a reason for hiding this comment

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

Nit then good!

docs/src/paper-wallet/installation.md Show resolved Hide resolved
@mergify mergify bot dismissed t-nelson’s stale review March 4, 2020 22:49

Pull request has been modified.

@garious garious added the automerge Merge this Pull Request automatically once CI passes label Mar 4, 2020
@solana-grimes solana-grimes merged commit 7eedff2 into solana-labs:master Mar 4, 2020
mergify bot pushed a commit that referenced this pull request Mar 4, 2020
automerge

(cherry picked from commit 7eedff2)
solana-grimes added a commit that referenced this pull request Mar 5, 2020
danpaul000 pushed a commit to danpaul000/solana that referenced this pull request Jul 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Installation instructions after usage instructions
3 participants