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

Add docs for the builtin programs #10359

Merged
merged 2 commits into from
Jun 2, 2020
Merged

Conversation

garious
Copy link
Contributor

@garious garious commented Jun 1, 2020

Problem

The builtin programs are currently well-documented in the source code, but those are difficult to find.

Summary of Changes

Add a doc that connects the dots. Linking to the Rust docs is less than ideal, but didn't want to duplicate any content and see one of the two bitrot. Also, although those are docs of Rust enums, is actually declarative enough that it is a decent language-agnostic description of the binary format.

@garious garious added the v1.2 label Jun 2, 2020
@garious garious added the automerge Merge this Pull Request automatically once CI passes label Jun 2, 2020
@solana-grimes
Copy link
Contributor

💔 Unable to automerge due to CI failure

@solana-grimes solana-grimes removed the automerge Merge this Pull Request automatically once CI passes label Jun 2, 2020
@garious garious added the automerge Merge this Pull Request automatically once CI passes label Jun 2, 2020
@solana-grimes
Copy link
Contributor

💔 Unable to automerge due to CI failure

@solana-grimes solana-grimes removed the automerge Merge this Pull Request automatically once CI passes label Jun 2, 2020
@garious garious merged commit 8c3ad85 into solana-labs:master Jun 2, 2020
mergify bot pushed a commit that referenced this pull request Jun 2, 2020
* Add docs for the builtin programs

* Cleanup

(cherry picked from commit 8c3ad85)
solana-grimes pushed a commit that referenced this pull request Jun 2, 2020
danpaul000 pushed a commit to danpaul000/solana that referenced this pull request Jul 8, 2020
* Add docs for the builtin programs

* Cleanup
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