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: deps: stop using go-libp2p deprecated peer.ID.Pretty #11263

Merged

Conversation

Icarus9913
Copy link
Contributor

@Icarus9913 Icarus9913 commented Sep 12, 2023

The go-libp2p peerID Pretty method is already deprecated, switch it to call String method.

Signed-off-by: Icarus9913 icaruswu66@qq.com

Checklist

Before you mark the PR ready for review, please make sure that:

  • Commits have a clear commit message.
  • PR title is in the form of of <PR type>: <area>: <change being made>
    • example: fix: mempool: Introduce a cache for valid signatures
    • PR type: fix, feat, build, chore, ci, docs, perf, refactor, revert, style, test
    • area, e.g. api, chain, state, market, mempool, multisig, networking, paych, proving, sealing, wallet, deps
  • If the PR affects users (e.g., new feature, bug fix, system requirements change), update the CHANGELOG.md and add details to the UNRELEASED section.
  • New features have usage guidelines and / or documentation updates in
  • Tests exist for new functionality or change in behavior
  • CI is green

@Icarus9913 Icarus9913 requested a review from a team as a code owner September 12, 2023 05:00
@Icarus9913
Copy link
Contributor Author

cc @marten-seemann

@marten-seemann
Copy link
Contributor

There seem to be more occurrences of this: https://github.com/search?q=repo%3Afilecoin-project%2Flotus%20Pretty()&type=code

Signed-off-by: Icarus9913 <icaruswu66@qq.com>
@Icarus9913 Icarus9913 force-pushed the libp2p/remove-deprecated-method branch from 0e1bf84 to 2a8ea95 Compare September 12, 2023 05:13
@Icarus9913
Copy link
Contributor Author

All cleaned for lotus.

@marten-seemann
Copy link
Contributor

@filecoin-project/lotus-maintainers For some context, we're planning to remove the deprecated method in the next release: libp2p/go-libp2p#2565. Merging this PR early will make updating to next release easier.

@magik6k magik6k merged commit 9999050 into filecoin-project:master Sep 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants