-
Notifications
You must be signed in to change notification settings - Fork 25
chore: remove unused dependencies in the workspace #732
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
Conversation
Manual Deploy AvailableYou can trigger a manual deploy of this PR branch to testnet: Alternative: Comment
Comment updated automatically when the PR is synchronized. |
📝 WalkthroughWalkthroughThis PR removes unused dependencies across the workspace and individual crates. Changes include removal of workspace-level dependencies (borsh-derive, cargo-lock, const_format, convert_case, hostname, itertools, proc-macro2, quote, serde_derive, and various Solana-related crates) and crate-specific dependencies (serde, magicblock-delegation-program, futures-util, solana-rpc-client, clap, and others). No new dependencies are added, and no public API declarations are modified. The net effect is a reduction in the overall dependency graph footprint across multiple crates. Possibly related PRs
Suggested reviewers
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (16)
💤 Files with no reviewable changes (16)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
GabrielePicco
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🚀
* master: feat: add program subscription support (#703) feat: add a way to remove confined accounts on start (#726) chore: remove unused dependencies in the workspace (#732) fix: prevent removal of delegated/undelegating accounts on eviction (#724) fix: Fix per-account livelock (#728) fix: properly prune program cache upon corruption (#730)

This PR removes all of the dangling dependencies throughout the primary workspace. The number of build dependencies has been reduced by 20% as a result of this cleanup.
Summary by CodeRabbit
Release Notes
✏️ Tip: You can customize this high-level summary in your review settings.