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

fix(libp2p): don't re-export libp2p-perf #3990

Merged
merged 2 commits into from
May 26, 2023
Merged

Conversation

mxinden
Copy link
Member

@mxinden mxinden commented May 26, 2023

Description

This commit removes the libp2p-perf protocol from libp2p crate. To use libp2p-perf one needs to import it directly. Rational beeing that libp2p-perf's API is not yet stable enough and will most likely require breaking changes in the near future.

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

This commit removes the `libp2p-perf` protocol from `libp2p` crate. To use `libp2p-perf` one needs
to import it directly. Rational beeing that `libp2p-perf`'s API is not yet stable enough and will
most likely require breaking changes in the near future.
@mxinden mxinden requested a review from thomaseizinger May 26, 2023 07:29
libp2p/CHANGELOG.md Outdated Show resolved Hide resolved
@mxinden
Copy link
Member Author

mxinden commented May 26, 2023

For the record, I initially advocated for including it in the libp2p meta crate.

@mergify mergify bot merged commit d4c4078 into libp2p:master May 26, 2023
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