Skip to content
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

Extract program-pack crate #2930

Merged
merged 4 commits into from
Oct 4, 2024
Merged

Conversation

kevinheavey
Copy link

Problem

The Pack trait imposes a solana_program requirement on many programs, most notably in the SPL

Summary of Changes

Move program_pack.rs to its own crate and re-export with deprecation

This branches off #2413 so that needs to be merged first

@kevinheavey kevinheavey force-pushed the extract-program-pack branch 2 times, most recently from 21928ff to 9055432 Compare September 17, 2024 01:17
@kevinheavey kevinheavey marked this pull request as ready for review September 24, 2024 08:10
@kevinheavey kevinheavey requested a review from febo October 1, 2024 21:51
sdk/program-pack/Cargo.toml Outdated Show resolved Hide resolved
Copy link

@febo febo left a comment

Choose a reason for hiding this comment

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

Looks very good, just a nit on the crate description.

@kevinheavey
Copy link
Author

@yihau CI failures are unrelated so I've sent a crate ownership invite to anza-team

febo
febo previously approved these changes Oct 3, 2024
sdk/program-pack/src/lib.rs Outdated Show resolved Hide resolved
@kevinheavey kevinheavey added the automerge automerge Merge this Pull Request automatically once CI passes label Oct 4, 2024
Copy link

mergify bot commented Oct 4, 2024

automerge label removed due to a CI failure

@mergify mergify bot removed the automerge automerge Merge this Pull Request automatically once CI passes label Oct 4, 2024
@kevinheavey kevinheavey added the automerge automerge Merge this Pull Request automatically once CI passes label Oct 4, 2024
@kevinheavey kevinheavey added automerge automerge Merge this Pull Request automatically once CI passes and removed automerge automerge Merge this Pull Request automatically once CI passes labels Oct 4, 2024
@kevinheavey
Copy link
Author

@febo could you merge this please? I don't have write access - usually automerge works for me but not today it seems

@febo febo merged commit 46a89da into anza-xyz:master Oct 4, 2024
53 checks passed
@kevinheavey kevinheavey deleted the extract-program-pack branch October 5, 2024 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge automerge Merge this Pull Request automatically once CI passes need:merge-assist
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants