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: update libp2p daemon with peer-store #36

Merged
merged 5 commits into from
Jun 8, 2020

Conversation

vasco-santos
Copy link
Member

@vasco-santos vasco-santos commented Apr 1, 2020

In the context of the PeerStore improvements specified on libp2p/js-libp2p#582 and as a follow up of libp2p/js-libp2p#590.

This PR contains two commits, one for updating the peer-store API and a second one to use the new libp2p API with no peer-info

The modules changed (libp2p and libp2p-kad-dht) were updated.

Needs:

@vasco-santos vasco-santos force-pushed the chore/update-libp2p-daemon-with-peerstore branch 3 times, most recently from dbc8ebd to b5e4229 Compare April 17, 2020 21:37
@vasco-santos vasco-santos force-pushed the chore/update-libp2p-daemon-with-peerstore branch from b5e4229 to ec1a1e0 Compare May 5, 2020 07:44
@vasco-santos vasco-santos force-pushed the chore/update-libp2p-daemon-with-peerstore branch from ec1a1e0 to 52c31c3 Compare May 13, 2020 09:03
@vasco-santos vasco-santos force-pushed the chore/update-libp2p-daemon-with-peerstore branch from 52c31c3 to 917286a Compare May 14, 2020 09:07
@vasco-santos vasco-santos requested a review from jacobheun June 8, 2020 08:25
Copy link
Contributor

@jacobheun jacobheun left a comment

Choose a reason for hiding this comment

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

LGTM 🚢

@vasco-santos vasco-santos merged commit 82c01bd into master Jun 8, 2020
@vasco-santos vasco-santos deleted the chore/update-libp2p-daemon-with-peerstore branch June 8, 2020 10:04
@vasco-santos vasco-santos restored the chore/update-libp2p-daemon-with-peerstore branch June 8, 2020 10:05
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.

2 participants