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 transaction-context crate #3132

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

kevinheavey
Copy link

Problem

solana_sdk::transaction_context imposes a solana_sdk dep on solana_transaction_status_client_types

Summary of Changes

  • Move to its own crate and re-export with deprecation notice
  • transaction_context.rs was making use of the full feature of solana-sdk for code using debug_assertions. Add a debug_assertions feature to the new crate to replace the full feature.
  • Make serde and bincode optional in the new crate
  • Add doc_auto_cfg like in add doc_auto_cfg to relevant sdk crates #3121

@kevinheavey kevinheavey force-pushed the extract-transaction-context branch 3 times, most recently from 6b718a4 to 04c3850 Compare October 15, 2024 16:15
@kevinheavey kevinheavey force-pushed the extract-transaction-context branch 3 times, most recently from a97f5ff to a2ca822 Compare October 22, 2024 18:42
@kevinheavey kevinheavey force-pushed the extract-transaction-context branch 3 times, most recently from 989ee76 to f30e79e Compare October 29, 2024 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

1 participant