-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Document solana_program::instruction #21817
Conversation
3229d6a
to
cdcd75c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple nits, otherwise looks great
Co-authored-by: Tyera Eulberg <teulberg@gmail.com>
Suggestions committed. Thanks for catching my typos. Great eyes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent, thank you!
Codecov Report
@@ Coverage Diff @@
## master #21817 +/- ##
=======================================
Coverage 81.3% 81.3%
=======================================
Files 516 516
Lines 144033 144033
=======================================
+ Hits 117139 117144 +5
+ Misses 26894 26889 -5 |
* Document solana_program::instruction * Apply suggestions from code review Co-authored-by: Tyera Eulberg <teulberg@gmail.com> Co-authored-by: Tyera Eulberg <teulberg@gmail.com> (cherry picked from commit dcb5849)
* Document solana_program::instruction * Apply suggestions from code review Co-authored-by: Tyera Eulberg <teulberg@gmail.com> Co-authored-by: Tyera Eulberg <teulberg@gmail.com> (cherry picked from commit dcb5849) Co-authored-by: Brian Anderson <andersrb@gmail.com>
This PR has been automatically locked since there has not been any activity in past 14 days after it was merged. |
Problem
Hackers love docs and they don't get enough of them.
Summary of Changes
This adds docs and examples for everything in the instruction module. Please review the "Specifying account metadata" section closely.
I made the
checked_add
function hidden as it seems to be intended for internal use.cc @askibin I appreciated your review of my last doc patch.