Skip to content

Cargo deny cleanup#65

Merged
mikalsande merged 4 commits intomainfrom
cargo_deny_cleanup
May 12, 2025
Merged

Cargo deny cleanup#65
mikalsande merged 4 commits intomainfrom
cargo_deny_cleanup

Conversation

@mikalsande
Copy link
Copy Markdown
Contributor

No description provided.

* Move all Lighthouse git crates to the same place
* Set shared dependencies with Lighthouse to the same version
  that Lighthouse uses.
* Update crates that are not directly Lighthouse or Ethereum
  related.
@mikalsande mikalsande self-assigned this May 12, 2025
Copy link
Copy Markdown

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 cleans up dependency management and updates random salt and seed generation by replacing deprecated getrandom::getrandom calls with getrandom::fill. Additionally, the update aligns Cargo.toml dependency versions and introduces explicit git source allowances in deny.toml.

Reviewed Changes

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

File Description
src/utils.rs Replaces getrandom::getrandom with getrandom::fill for key derivation.
src/seed.rs Updates seed generation to use getrandom::fill.
deny.toml Adds explicit git source allowance for Lighthouse dependencies.
Cargo.toml Updates dependency versions (e.g., reqwest, getrandom, ethereum packages) and enables resolver "2".

Comment thread deny.toml
Copy link
Copy Markdown
Contributor

@mariavarvaroi mariavarvaroi left a comment

Choose a reason for hiding this comment

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

LGTM!

@mikalsande mikalsande merged commit e330329 into main May 12, 2025
2 checks passed
@mariavarvaroi mariavarvaroi deleted the cargo_deny_cleanup branch May 12, 2025 13:43
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