Skip to content

fix: specify correct libp2p-swarm dependency#3711

Merged
mergify[bot] merged 2 commits intomasterfrom
fix/swarm-dependency
Apr 5, 2023
Merged

fix: specify correct libp2p-swarm dependency#3711
mergify[bot] merged 2 commits intomasterfrom
fix/swarm-dependency

Conversation

@thomaseizinger
Copy link
Copy Markdown
Contributor

Description

With #3658, these crates depend on the 0.42.1 release to access the new ToSwarm type. With the currently specified version, a user could theoretically run into a compile error if they pin libp2p-swarm to 0.42.0 in their lockfile but update to the latest patch release of one of these crates.

Notes & open questions

Change checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • A changelog entry has been made in the appropriate crates

Copy link
Copy Markdown
Member

@mxinden mxinden left a comment

Choose a reason for hiding this comment

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

Thank you!

@mxinden mxinden added the send-it label Apr 5, 2023
@mxinden
Copy link
Copy Markdown
Member

mxinden commented Apr 5, 2023

Thus far I had the impression that mergify would automatically merge master when I add the send-it label. Was that never the case @thomaseizinger?

@mergify mergify bot merged commit 451c64a into master Apr 5, 2023
@mergify mergify bot deleted the fix/swarm-dependency branch April 5, 2023 19:44
umgefahren pushed a commit to umgefahren/rust-libp2p that referenced this pull request Mar 8, 2024
With libp2p#3658, these crates depend on the `0.42.1` release to access the new `ToSwarm` type. With the currently specified version, a user could theoretically run into a compile error if they pin `libp2p-swarm` to `0.42.0` in their lockfile but update to the latest patch release of one of these crates.

Pull-Request: libp2p#3711.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants