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

Update libp2p to 0.35 #928

Merged
merged 14 commits into from
Feb 23, 2021
Merged

Update libp2p to 0.35 #928

merged 14 commits into from
Feb 23, 2021

Conversation

austinabell
Copy link
Contributor

Summary of changes
Changes introduced in this pull request:

  • Includes gossipsub 1.1 changes
  • Currently broken when syncing, upgrades fail

Just going to keep as draft until I have more time to sink into debugging the issues

Reference issue to close (if applicable)

Closes

Other information and links

@austinabell austinabell changed the title Update libp2p Update libp2p to 0.34 Jan 13, 2021
Base automatically changed from austin/depupd to main January 14, 2021 17:22
@cryptoquick
Copy link
Contributor

I'm having trouble with updating the lockfile because of work being done in this PR.

See:

@ec2 ec2 changed the title Update libp2p to 0.34 Update libp2p to 0.35 Feb 22, 2021
@ec2 ec2 marked this pull request as ready for review February 22, 2021 17:37
Copy link
Contributor

@cryptoquick cryptoquick left a comment

Choose a reason for hiding this comment

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

This is a lot of good work! Only one small nitpick, feel free to ignore if you like. I like how you were able to minimize a few clones, too.

node/forest_libp2p/src/behaviour.rs Show resolved Hide resolved
@ec2 ec2 merged commit f6eddd5 into main Feb 23, 2021
@ec2 ec2 deleted the austin/libp2p34 branch February 23, 2021 16:16
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.

None yet

3 participants