Skip to content

Comments

pod!: Replace Pubkey w/ Address in spl-pod#177

Merged
grod220 merged 2 commits intomainfrom
spl-pod-address
Feb 25, 2026
Merged

pod!: Replace Pubkey w/ Address in spl-pod#177
grod220 merged 2 commits intomainfrom
spl-pod-address

Conversation

@grod220
Copy link
Member

@grod220 grod220 commented Feb 24, 2026

solana-pubkey migration to solana-address contributing to the no-std effort for spl-pod.

#[repr(transparent)]
pub struct OptionalNonZeroPubkey(pub Pubkey);
impl TryFrom<Option<Pubkey>> for OptionalNonZeroPubkey {
pub struct OptionalNonZeroPubkey(pub Address);
Copy link
Member Author

Choose a reason for hiding this comment

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

Not going to rename OptionalNonZeroPubkey as we have an upcoming update to migrate this functionality to PodOption

@grod220 grod220 mentioned this pull request Feb 24, 2026
7 tasks
@grod220 grod220 requested review from febo and joncinque February 24, 2026 21:28
Copy link
Contributor

@joncinque joncinque left a comment

Choose a reason for hiding this comment

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

Looks great! Just one question on the features

Copy link
Contributor

@joncinque joncinque left a comment

Choose a reason for hiding this comment

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

Looks great!

@joncinque joncinque changed the title Replace Pubkey w/ Address in spl-pod pod: Replace Pubkey w/ Address in spl-pod Feb 24, 2026
@grod220 grod220 merged commit b2d7533 into main Feb 25, 2026
44 checks passed
@grod220 grod220 deleted the spl-pod-address branch February 25, 2026 09:48
@grod220 grod220 changed the title pod: Replace Pubkey w/ Address in spl-pod pod!: Replace Pubkey w/ Address in spl-pod Feb 25, 2026
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