Skip to content

program: bump deps to SDK 3.0#88

Merged
buffalojoec merged 2 commits intomainfrom
bump-deps-sdk-3.0
Oct 7, 2025
Merged

program: bump deps to SDK 3.0#88
buffalojoec merged 2 commits intomainfrom
bump-deps-sdk-3.0

Conversation

@buffalojoec
Copy link
Copy Markdown
Contributor

Does the whole "busting up" of solana-program and solana-sdk wherever necessary, upgrading the program up to use the isolated crates on SDK 3.0.

Copy link
Copy Markdown
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.

Just a tiny thing on the config interface, the rest looks great!

Comment thread program/src/lib.rs
#[cfg(feature = "bpf-entrypoint")]
pub mod entrypoint;

pub use solana_program;
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.

We could eventually re-export some core types, but I haven't found that super useful in the past, so I'm happy to have this removal

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sounds good to me! ✂️

Comment thread program/Cargo.toml Outdated
solana-sysvar = { version = "2.2.1", features = ["bincode"] }
rand = "0.8.5"
solana-account = { version = "3.0.0", features = ["bincode"] }
solana-config-interface = { version = "1", features = ["serde"] }
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.

It looks like the build is ok, but this should probably be v2 to be consistent

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done!

Copy link
Copy Markdown
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!

@buffalojoec buffalojoec merged commit cf77d09 into main Oct 7, 2025
9 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