Skip to content

Create network_utils crate #7761

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: unstable
Choose a base branch
from
Open

Conversation

dknopik
Copy link
Member

@dknopik dknopik commented Jul 18, 2025

Issue Addressed

Anchor currently depends on lighthouse_network for a few types and utilities that live within. As we use our own libp2p behaviours, we actually do not use the core logic in that crate. This makes us transitively depend on a bunch of unneeded crates (even a whole separate libp2p if the versions mismatch!)

Proposed Changes

Move things we require into it's own lightweight crate.

Additional Info

CC @diegomrsantos, wdyt?

@dknopik dknopik requested a review from jxs as a code owner July 18, 2025 14:53
@dknopik dknopik changed the title Crate network_utils crate Create network_utils crate Jul 18, 2025
@macladson
Copy link
Member

Love this idea! Just needs a cargo sort --workspace and a cargo fmt to pass CI

@macladson macladson added waiting-on-author The reviewer has suggested changes and awaits thier implementation. code-quality labels Jul 22, 2025
@mergify mergify bot added ready-for-review The code is ready for review and removed waiting-on-author The reviewer has suggested changes and awaits thier implementation. labels Jul 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code-quality ready-for-review The code is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants