Skip to content
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

chore(examples): remove unused dependency from rendezvous #5580

Merged
merged 2 commits into from
Aug 30, 2024

Conversation

P1R
Copy link
Contributor

@P1R P1R commented Aug 29, 2024

Description

Following on issue #4449
refactor: remove unnecesary dependencies rendezvous example (async-std)

Notes & open questions

in the rendezvous example there where unnecessary dependencies in the Cargo.toml

Change checklist

  • Removed unnecessary dependencies on examples/rendezvous/Cargo.toml
  • 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

@P1R P1R changed the title refactor: remove unnecessary dependencies rendezvous example (async-std) refactor(examples): remove unnecessary dependencies rendezvous (async-std) Aug 29, 2024
@dariusc93 dariusc93 changed the title refactor(examples): remove unnecessary dependencies rendezvous (async-std) chore(examples): remove unusedf dependency from rendezvous Aug 30, 2024
@dariusc93 dariusc93 changed the title chore(examples): remove unusedf dependency from rendezvous chore(examples): remove unused dependency from rendezvous Aug 30, 2024
Copy link
Member

@jxs jxs left a comment

Choose a reason for hiding this comment

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

LGTM, thanks David!

@jxs jxs added the send-it label Aug 30, 2024
@mergify mergify bot merged commit f38fb5c into libp2p:master Aug 30, 2024
72 checks passed
TimTinkers pushed a commit to unattended-backpack/rust-libp2p that referenced this pull request Sep 14, 2024
## Description

<!--
Please write a summary of your changes and why you made them.
This section will appear as the commit message after merging.
Please craft it accordingly.
For a quick primer on good commit messages, check out this blog post:
https://cbea.ms/git-commit/

Please include any relevant issues in here, for example:

Related https://github.com/libp2p/rust-libp2p/issues/ABCD.
Fixes https://github.com/libp2p/rust-libp2p/issues/XYZ.
-->
Following on issue libp2p#4449 
refactor: remove unnecesary dependencies rendezvous example (async-std)
## Notes & open questions

<!--
Any notes, remarks or open questions you have to make about the PR which
don't need to go into the final commit message.
-->
in the rendezvous example there where unnecessary dependencies in the
Cargo.toml

## Change checklist

* Removed unnecessary dependencies on examples/rendezvous/Cargo.toml

<!-- Please add a Changelog entry in the appropriate crates and bump the
crate versions if needed. See
<https://github.com/libp2p/rust-libp2p/blob/master/docs/release.md#development-between-releases>-->

- [x] 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

Co-authored-by: David E. Perez Negron R. <david@nethunters.xyz>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
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