Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Expand docs for Pubkey::create_program_address#21750

Merged
mergify[bot] merged 2 commits intosolana-labs:masterfrom
brson:docs-create-program-address
Dec 9, 2021
Merged

Expand docs for Pubkey::create_program_address#21750
mergify[bot] merged 2 commits intosolana-labs:masterfrom
brson:docs-create-program-address

Conversation

@brson
Copy link
Copy Markdown
Contributor

@brson brson commented Dec 9, 2021

Problem

The create_program_address docs say that it has no use, but it is useful for verifying PDAs. re #21528 (comment)

Summary of Changes

Expand the docs to explain how to use create_program_address.

cc @askibin

@mergify mergify Bot added the community Community contribution label Dec 9, 2021
@mergify mergify Bot requested a review from a team December 9, 2021 21:18
CriesofCarrots
CriesofCarrots previously approved these changes Dec 9, 2021
Copy link
Copy Markdown
Contributor

@CriesofCarrots CriesofCarrots left a comment

Choose a reason for hiding this comment

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

Ah, thanks so much for researching this! I wasn't clued in to this use-case.
Asking Jack to confirm, but lgtm.

Comment thread sdk/program/src/pubkey.rs Outdated
jackcmay
jackcmay previously approved these changes Dec 9, 2021
@mergify mergify Bot dismissed stale reviews from CriesofCarrots and jackcmay December 9, 2021 21:46

Pull request has been modified.

@CriesofCarrots CriesofCarrots added the automerge Merge this Pull Request automatically once CI passes label Dec 9, 2021
@mergify mergify Bot requested a review from a team December 9, 2021 21:46
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 9, 2021

Codecov Report

Merging #21750 (3109e34) into master (6fc3291) will decrease coverage by 0.0%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #21750     +/-   ##
=========================================
- Coverage    81.4%    81.4%   -0.1%     
=========================================
  Files         511      511             
  Lines      143469   143469             
=========================================
- Hits       116840   116827     -13     
- Misses      26629    26642     +13     

@mergify mergify Bot merged commit 6919c48 into solana-labs:master Dec 9, 2021
mergify Bot pushed a commit that referenced this pull request Dec 9, 2021
* Expand docs for Pubkey::create_program_address

* Update sdk/program/src/pubkey.rs

Co-authored-by: Tyera Eulberg <teulberg@gmail.com>
(cherry picked from commit 6919c48)
jstarry pushed a commit that referenced this pull request Dec 10, 2021
* Expand docs for Pubkey::create_program_address

* Update sdk/program/src/pubkey.rs

Co-authored-by: Tyera Eulberg <teulberg@gmail.com>
(cherry picked from commit 6919c48)

Co-authored-by: Brian Anderson <andersrb@gmail.com>
@brooksprumo brooksprumo mentioned this pull request Jan 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

automerge Merge this Pull Request automatically once CI passes community Community contribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants