Skip to content

chore: remove unused workspace dependencies - #778

Merged
Evgenii (Vaiz) merged 2 commits into
mainfrom
u/eshutov/remove-unused-workspace-deps
Sep 25, 2026
Merged

Evgenii (Vaiz) merged 2 commits into
mainfrom
u/eshutov/remove-unused-workspace-deps

Conversation

@Vaiz

@Vaiz Evgenii (Vaiz) commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

🤖 Clawpilot here! Posted automatically by Clawpilot (an AI agent), not by a human. Please verify before acting.

What this changes

Cargo.toml no longer declares five unused external workspace dependencies.

Details

  • Remove anyhow, pbjson-types, pct-str, pretty_assertions, and thiserror from [workspace.dependencies].
  • Preserve local crate entries in the workspace manifest.

Effects

  • Crate dependency graphs do not change because no crate inherited the removed entries.
  • anyhow remains transitive through bolero-engine and prost-derive.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings September 25, 2026 12:24

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

No unresolved issues were identified.

Review effort: Lite
Findings: None

What changed in this PR

Removes 12 unused workspace dependencies from Cargo.toml without affecting active dependency inheritance.

Changes:

  • Deletes unused external and internal workspace dependencies.
  • Retains all dependencies still referenced by workspace crates.
File Description
Cargo.toml Removes unused workspace dependency declarations.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings September 25, 2026 12:27

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Reconcile the PR description with the five dependencies actually removed.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 Low severity

Open (1)

Comment thread Cargo.toml
@Vaiz
Evgenii (Vaiz) enabled auto-merge (squash) September 25, 2026 12:31
@codecov

codecov Bot commented Sep 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.0%. Comparing base (3bd1aa4) to head (ff3e8d8).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #778   +/-   ##
=======================================
  Coverage   100.0%   100.0%           
=======================================
  Files         737      736    -1     
  Lines       98509    98426   -83     
=======================================
- Hits        98509    98426   -83     
Flag Coverage Δ
linux 100.0% <ø> (ø)
linux-arm 100.0% <ø> (ø)
scheduled ?
windows 100.0% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Vaiz
Evgenii (Vaiz) merged commit ca3a372 into main Sep 25, 2026
39 checks passed
@Vaiz
Evgenii (Vaiz) deleted the u/eshutov/remove-unused-workspace-deps branch September 25, 2026 16:44
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.

6 participants