Skip to content

Modernize program to use the Address type#73

Merged
lorisleiva merged 2 commits into
mainfrom
loris/modernize-program
Jun 4, 2026
Merged

Modernize program to use the Address type#73
lorisleiva merged 2 commits into
mainfrom
loris/modernize-program

Conversation

@lorisleiva

Copy link
Copy Markdown
Member

This PR modernizes the Feature Gate program to the solana-address era, replacing solana-pubkey with solana-address and bumping the program's Solana crates (and mollusk-svm) to current versions. The workspace toolchain and Solana CLI are bumped accordingly (the new dependencies require rustc 1.89).

This aligns feature-gate with the converged solana-program repos and is a prerequisite for regenerating the client with the latest Codama renderer.

The revoke_pending_activation compute-unit assertion is updated from 2777 to 2723, reflecting the slightly lower cost after the toolchain and dependency updates.

@lorisleiva lorisleiva force-pushed the loris/modernize-program branch from 24d5c5d to adfe35f Compare June 4, 2026 09:17
Modernizes the Feature Gate program to the solana-address era, replacing solana-pubkey with solana-address and bumping the program's Solana crates (and mollusk-svm) to current versions. The workspace toolchain and Solana CLI are bumped accordingly (the new dependencies require rustc 1.89). This aligns feature-gate with the converged solana-program repos and is a prerequisite for regenerating the client with the latest Codama renderer.
@lorisleiva lorisleiva force-pushed the loris/modernize-program branch from adfe35f to 1ed1c89 Compare June 4, 2026 09:25
@lorisleiva lorisleiva marked this pull request as ready for review June 4, 2026 09:35

@joncinque joncinque left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great overall! Just a comment on the dev dependencies for the address crate

Comment thread program/Cargo.toml Outdated
@lorisleiva lorisleiva requested a review from joncinque June 4, 2026 13:44

@joncinque joncinque left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great!

@lorisleiva lorisleiva merged commit 867fb23 into main Jun 4, 2026
22 checks passed
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