Skip to content

Comments

Update crates version#67

Merged
genedna merged 3 commits intoweb3infra-foundation:mainfrom
genedna:main
Nov 25, 2025
Merged

Update crates version#67
genedna merged 3 commits intoweb3infra-foundation:mainfrom
genedna:main

Conversation

@genedna
Copy link
Member

@genedna genedna commented Nov 25, 2025

No description provided.

Signed-off-by: Quanyi Ma <eli@patch.sh>
Copilot AI review requested due to automatic review settings November 25, 2025 05:38
Copy link
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

This PR updates dependency versions in the libra crate, removing many unused dependencies and updating the remaining ones to specific semantic versions. The changes include:

  • Dependency cleanup: Removed ~60 unused dependencies (e.g., ahash, axum, bincode, dashmap, encoding_rs, openssl, quickcheck, rayon, etc.)
  • Version updates: Updated all remaining dependencies to fully specified versions (e.g., anyhow from "1.0" to "1.0.100")
  • Feature additions: Added "process" feature to tokio in dev-dependencies

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
Cargo.toml Removed ~60 unused dependencies and updated remaining dependencies to fully specified versions with proper formatting
Cargo.lock Automatically regenerated to reflect the Cargo.toml changes, removing transitive dependencies of removed crates

Signed-off-by: Quanyi Ma <eli@patch.sh>
Copy link
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

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

}

// Fallback 3: root directory to ensure a stable, existing path
PathBuf::from("/")
Copy link

Copilot AI Nov 25, 2025

Choose a reason for hiding this comment

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

The fallback path / (root directory) on line 56 is problematic. This would be inappropriate as a working directory on most systems due to permission constraints, and could lead to unexpected behavior. Consider using a more appropriate fallback such as the user's home directory or a temp directory.

Copilot uses AI. Check for mistakes.
Signed-off-by: Quanyi Ma <eli@patch.sh>
@genedna genedna merged commit 76948ef into web3infra-foundation:main Nov 25, 2025
5 checks passed
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.

1 participant