Skip to content

rpc-client-api: reexport publicly used types#7808

Merged
joncinque merged 2 commits intoanza-xyz:masterfrom
kevinheavey:rpc-client-api-reexports
Sep 2, 2025
Merged

rpc-client-api: reexport publicly used types#7808
joncinque merged 2 commits intoanza-xyz:masterfrom
kevinheavey:rpc-client-api-reexports

Conversation

@kevinheavey
Copy link
Copy Markdown

Similar to #7737

@mergify
Copy link
Copy Markdown

mergify Bot commented Aug 31, 2025

If this PR represents a change to the public RPC API:

  1. Make sure it includes a complementary update to rpc-client/ (example)
  2. Open a follow-up PR to update the JavaScript client @solana/kit (example)

Thank you for keeping the RPC clients in sync with the server API @kevinheavey.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Aug 31, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.0%. Comparing base (7eb0faa) to head (70dbbe9).
⚠️ Report is 2348 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #7808   +/-   ##
=======================================
  Coverage    83.0%    83.0%           
=======================================
  Files         812      812           
  Lines      357194   357194           
=======================================
+ Hits       296736   296818   +82     
+ Misses      60458    60376   -82     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown

@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! Confirming the rest of the crate is properly re-exporting publicly used types

@joncinque joncinque merged commit 5bc175a into anza-xyz:master Sep 2, 2025
44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants