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

refactor: stop using go-libp2p deprecated peer.ID.Pretty #10121

Merged
merged 1 commit into from
Sep 18, 2023

Conversation

Icarus9913
Copy link
Contributor

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

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

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

cc @marten-seemann

.gitignore Outdated Show resolved Hide resolved
p2p/local.go Outdated Show resolved Hide resolved
@Icarus9913 Icarus9913 force-pushed the libp2p/remove-deprecated-method branch from 92da467 to 9a96694 Compare September 12, 2023 04:53
@marten-seemann
Copy link
Member

I see 19 files that use Pretty() in this repo: https://github.com/search?q=repo%3Aipfs%2Fkubo%20Pretty()&type=code. @Icarus9913 did you miss some occurrences?

@Icarus9913 Icarus9913 force-pushed the libp2p/remove-deprecated-method branch from 9a96694 to 1b77f2c Compare September 12, 2023 05:19
@Icarus9913
Copy link
Contributor Author

I see 19 files that use Pretty() in this repo: https://github.com/search?q=repo%3Aipfs%2Fkubo%20Pretty()&type=code. @Icarus9913 did you miss some occurrences?

All cleaned

@Icarus9913 Icarus9913 force-pushed the libp2p/remove-deprecated-method branch 2 times, most recently from ec4831c to 0058a81 Compare September 12, 2023 05:26
@marten-seemann
Copy link
Member

@ipfs/kubo-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.

core/commands/swarm.go Outdated Show resolved Hide resolved
Signed-off-by: Icarus9913 <icaruswu66@qq.com>
@hacdias hacdias changed the title stop using go-libp2p deprecated peer.ID.Pretty refactor: stop using go-libp2p deprecated peer.ID.Pretty Sep 18, 2023
@hacdias hacdias added the skip/changelog This change does NOT require a changelog entry label Sep 18, 2023
@hacdias hacdias merged commit 9fdb085 into ipfs:master Sep 18, 2023
18 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip/changelog This change does NOT require a changelog entry
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants