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 inflation crate #2970

Merged
merged 5 commits into from
Oct 28, 2024
Merged

Conversation

kevinheavey
Copy link

Problem

solana_sdk::inflation imposes a solana-sdk dependency on solana-rpc-client-api among other things.

Summary of Changes

  • Move to its own crate
  • Re-export with deprecation notice
  • Make serde optional in the new crate

@kevinheavey kevinheavey force-pushed the extract-inflation branch 2 times, most recently from 7f9fb8a to 2663e0b Compare September 26, 2024 09:35
@kevinheavey kevinheavey force-pushed the extract-inflation branch 4 times, most recently from 48b6f30 to 31d28ec Compare October 10, 2024 22:41
@kevinheavey kevinheavey force-pushed the extract-inflation branch 3 times, most recently from 52e8bae to 6d66ed1 Compare October 22, 2024 10:20
buffalojoec
buffalojoec previously approved these changes Oct 28, 2024
@kevinheavey
Copy link
Author

@buffalojoec had to rebase to fix conflicts, could you approve again please?

@yihau I've sent an ownership invite for solana-inflation

@kevinheavey kevinheavey added the automerge automerge Merge this Pull Request automatically once CI passes label Oct 28, 2024
@mergify mergify bot merged commit 773fe22 into anza-xyz:master Oct 28, 2024
53 checks passed
ray-kast pushed a commit to abklabs/agave that referenced this pull request Nov 27, 2024
* extract inflation crate

* fix frozen-abi stuff

* missing symbol

* update digest

* update digest
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
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants