Skip to content

rpc: Remove unused solana-sdk dependency#6301

Merged
mergify[bot] merged 1 commit intoanza-xyz:masterfrom
steviez:rm_moar_sdk
May 23, 2025
Merged

rpc: Remove unused solana-sdk dependency#6301
mergify[bot] merged 1 commit intoanza-xyz:masterfrom
steviez:rm_moar_sdk

Conversation

@steviez
Copy link
Copy Markdown

@steviez steviez commented May 23, 2025

Problem

@t-nelson already did all of the actual work in removing use of solana-sdk usage from solana-rpc, this just removes solana-sdk from the dependency list in Cargo.toml

@steviez steviez requested a review from t-nelson May 23, 2025 17:07
@steviez steviez added the automerge automerge Merge this Pull Request automatically once CI passes label May 23, 2025
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.7%. Comparing base (f3a45e7) to head (47a7f36).

Additional details and impacted files
@@            Coverage Diff            @@
##           master    #6301     +/-   ##
=========================================
- Coverage    82.8%    82.7%   -0.1%     
=========================================
  Files         845      845             
  Lines      377826   377826             
=========================================
- Hits       312851   312809     -42     
- Misses      64975    65017     +42     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mergify mergify Bot merged commit 7a5006d into anza-xyz:master May 23, 2025
59 checks passed
@steviez steviez deleted the rm_moar_sdk branch May 23, 2025 17:34
@steviez steviez mentioned this pull request May 27, 2025
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants