Skip to content

dependencies clean up#903

Merged
mskrzypkows merged 1 commit intomasterfrom
dep_clean_up
Mar 12, 2026
Merged

dependencies clean up#903
mskrzypkows merged 1 commit intomasterfrom
dep_clean_up

Conversation

@mskrzypkows
Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings March 12, 2026 10:00
@mskrzypkows mskrzypkows enabled auto-merge (squash) March 12, 2026 10:00
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Cleans up the workspace dependency graph by removing unused direct crate dependencies and tightening the enabled alloy feature set at the workspace level.

Changes:

  • Remove unused direct dependencies from shasta and pacaya crate manifests.
  • Remove lazy_static from common and the workspace dependency list.
  • Replace alloy’s broad feature enablement with an explicit, narrower feature allowlist and update Cargo.lock accordingly.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
shasta/Cargo.toml Drops unused direct dependencies to rely on transitive/workspace deps only where needed.
pacaya/Cargo.toml Drops unused direct dependencies while keeping required ones like alloy-rlp.
common/Cargo.toml Removes lazy_static from direct dependencies.
Cargo.toml Tightens alloy workspace features and removes lazy_static from workspace deps.
Cargo.lock Reflects the updated resolved dependency graph after the cleanup.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@mskrzypkows mskrzypkows merged commit 6ee791e into master Mar 12, 2026
12 checks passed
@mskrzypkows mskrzypkows deleted the dep_clean_up branch March 12, 2026 10:05
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