Skip to content
This repository has been archived by the owner on Apr 29, 2020. It is now read-only.

Advanced peer-book #91

Merged
merged 8 commits into from
Mar 31, 2017
Merged

Advanced peer-book #91

merged 8 commits into from
Mar 31, 2017

Conversation

daviddias
Copy link
Collaborator

@daviddias daviddias commented Mar 29, 2017 •
edited
Loading

@daviddias
Copy link
Collaborator Author

Once this is done, bubble up the changes in the dependents chain :)

@dignifiedquire dignifiedquire mentioned this pull request Mar 29, 2017
Merged
38 tasks
@daviddias
Copy link
Collaborator Author

daviddias commented Mar 30, 2017 •
edited
Loading

Ok, update on this. Going to update the PeerId API and then start bubbling up all the changes.

Doing the following in a next PR:

  • update swarm to take peerBook so that it can update the state of each peerInfo

dignifiedquire
dignifiedquire approved these changes Mar 31, 2017
View reviewed changes
Copy link
Contributor

@dignifiedquire dignifiedquire left a comment

Choose a reason for hiding this comment

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

code changes look, haven't closely checked the tests but they seem to be getting more so 👍

@daviddias daviddias merged commit a509052 into master Mar 31, 2017
@daviddias daviddias deleted the feat/advanced-peer-book branch March 31, 2017 15:55
@qmutz qmutz mentioned this pull request Feb 11, 2024
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@dignifiedquire dignifiedquire dignifiedquire approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@daviddias @dignifiedquire