Skip to content

transaction: re-export types used in public API#296

Merged
joncinque merged 5 commits intoanza-xyz:masterfrom
kevinheavey:transaction-reexports
Aug 26, 2025
Merged

transaction: re-export types used in public API#296
joncinque merged 5 commits intoanza-xyz:masterfrom
kevinheavey:transaction-reexports

Conversation

@kevinheavey
Copy link
Copy Markdown
Contributor

and switch to solana-address

@kevinheavey kevinheavey requested a review from a team as a code owner August 26, 2025 07:17
Copy link
Copy Markdown
Collaborator

@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! Just the question about InstructionError

Comment thread transaction/src/lib.rs
Copy link
Copy Markdown
Contributor

@apfitzge apfitzge left a comment

Choose a reason for hiding this comment

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

Perfect - thanks!

@apfitzge
Copy link
Copy Markdown
Contributor

Also just want to say I appreciate your clean commits on these, it does make it easier to review!

@joncinque joncinque merged commit b9551a3 into anza-xyz:master Aug 26, 2025
49 of 50 checks passed
febo pushed a commit to febo/solana-sdk that referenced this pull request Sep 21, 2025
* re-export essential types in solana-transaction

* re-export transaction error types and don't alias TransactionResult in crate

* re-export solana-signer items

* switch to solana-address

* fmt
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.

3 participants