Skip to content

ref(core): Cleanup unused dependencies and update - #642

Merged
iambriccardo merged 3 commits into
mainfrom
cleanup-deps
Mar 26, 2026
Merged

ref(core): Cleanup unused dependencies and update#642
iambriccardo merged 3 commits into
mainfrom
cleanup-deps

Conversation

@iambriccardo

@iambriccardo iambriccardo commented Mar 26, 2026

Copy link
Copy Markdown
Contributor

This PR cleans up unused dependencies and updates the Cargo.lock.

@coveralls

coveralls commented Mar 26, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 77.699% (-0.04%) from 77.735%
when pulling d61d596 on cleanup-deps
into 0e18d8f on main.

@iambriccardo iambriccardo changed the title ref(core): Cleanup unused dependencies ref(core): Cleanup unused dependencies and update Mar 26, 2026
@iambriccardo
iambriccardo marked this pull request as ready for review March 26, 2026 11:24
@iambriccardo
iambriccardo requested a review from a team as a code owner March 26, 2026 11:24
Copilot AI review requested due to automatic review settings March 26, 2026 11:24

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Removes unused Rust dependencies from workspace/crates and refreshes the lockfile to reflect the cleaned dependency graph across the ETL workspace.

Changes:

  • Drop unused serde_yaml dev-dependency from etl-config.
  • Drop unused etl dependency from etl-api and remove unused workspace dependency aliases (etl-api, etl-replicator, ring, serde_yaml).
  • Regenerate Cargo.lock to reflect dependency removals and crate updates.

Reviewed changes

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

File Description
etl-config/Cargo.toml Removes unused dev-dependency (serde_yaml).
etl-api/Cargo.toml Removes unused direct dependency on etl.
Cargo.toml Removes unused workspace dependency entries (and unused third-party workspace deps).
Cargo.lock Updates lockfile for the new dependency graph and bumped crate versions.

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

@iambriccardo
iambriccardo merged commit cc6a5d0 into main Mar 26, 2026
16 checks passed
@iambriccardo
iambriccardo deleted the cleanup-deps branch March 26, 2026 13:00
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.

4 participants