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(core/choice): log on dial and fix listen log #4164

Merged
merged 2 commits into from
Jul 10, 2023

Conversation

mxinden
Copy link
Member

@mxinden mxinden commented Jul 10, 2023

Description

  • Fixes typo in <OrTransport as Transport>::listen_on using the word "dial" instead of "listen_on".
  • Adds logging to <OrTransport as Transport>::dial.

Follow-up to #4133

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

- Fixes typo in `<OrTransport as Transport>::listen_on` using the word "dial" instead of "listen_on".
- Adds logging to `<OrTransport as Transport>::dial`.

Follow-up to libp2p#4133
@mxinden mxinden added send-it trivial Marks PRs which are considered trivial and don't need approval from another maintainer. labels Jul 10, 2023
@mergify mergify bot merged commit 2671375 into libp2p:master Jul 10, 2023
64 checks passed
thomaseizinger pushed a commit that referenced this pull request Aug 20, 2023
- Fixes typo in `<OrTransport as Transport>::listen_on` using the word "dial" instead of "listen_on".
- Adds logging to `<OrTransport as Transport>::dial`.

Follow-up to #4133

Pull-Request: #4164.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
send-it trivial Marks PRs which are considered trivial and don't need approval from another maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant