-
Notifications
You must be signed in to change notification settings - Fork 319
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 feature-set crate #2172
Conversation
If this PR represents a change to the public RPC API:
Thank you for keeping the RPC clients in sync with the server API @kevinheavey. |
55d2f0b
to
17fa98b
Compare
7c014d6
to
4ef520a
Compare
e154f71
to
e4ca26e
Compare
The Firedancer team maintains a line-for-line reimplementation of the |
d35364e
to
8b70ede
Compare
e02b542
to
075967f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just nits, looks great!
clock::Slot, | ||
epoch_schedule::EpochSchedule, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Once EpochSchedule gets moved out of solana_program
, this crate could depend only on pubkey, hash, clock, and that 🥳
4c901a2
to
d2a356d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This one also looks good, thanks! @yihau can you accept the crate ownership for solana-feature-set? Once checks pass, we can get this in
all green! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good to go!
* extract feature-set crate * update feature_set dependents * re-export with deprecation notice * fix path after rebase * fmt * fmt after rebase * fixes after rebase * unused import * fix import * unused import * post-rebase fixes * fix path * missing frozen-abi support * update lock file * use workspace lints * fix paths * fix import * remove unused import * remove unused deps * consolidate imports * add back dep (not unused after all)
Problem
solana_sdk::feature_set
blocks a lot of things from being pulled out of the sdk, includingsdk::transaction
Summary of Changes