Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

feat: use pull-mplex #1884

Merged
merged 1 commit into from
May 8, 2019
Merged

feat: use pull-mplex #1884

merged 1 commit into from
May 8, 2019

Conversation

jacobheun
Copy link
Contributor

@jacobheun jacobheun commented Feb 18, 2019

  • run ipfs/interop tests against this change

@ghost ghost assigned jacobheun Feb 18, 2019
@ghost ghost added the status/in-progress In progress label Feb 18, 2019
@hugomrdias
Copy link
Member

hugomrdias commented Apr 2, 2019

@jacobheun any chance i can help out with this ? this PR fixes vue support and protoschool

@jacobheun
Copy link
Contributor Author

@hugomrdias i just rebased this with master so it has the latest updates. I have a WIP pr over at ipfs/interop#59 that I haven't been able to spend a ton of time on. I think this is in a good spot, but the interop tests have been unreliable as of late. It would be nice to get more deterministic results there.

@alanshaw
Copy link
Member

@jacobheun can we ship this? 🚢 😁 🙏

@jacobheun
Copy link
Contributor Author

@alanshaw not yet, ipfs/interop has an issue with pubsub I did not have a chance to look at last week. I will look into it more today/tomorrow.

test: shut off dht by default
@jacobheun jacobheun changed the title [WIP] use pull-mplex feat: use pull-mplex May 6, 2019
@jacobheun jacobheun requested a review from alanshaw May 6, 2019 11:54
@jacobheun
Copy link
Contributor Author

The underlying issue with pubsub interop has been fixed, this is ready to go. You can see the passing interop tests at ipfs/interop#67.

Copy link
Member

@vasco-santos vasco-santos left a comment

Choose a reason for hiding this comment

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

LGTM

@alanshaw alanshaw merged commit bd3ade6 into master May 8, 2019
@ghost ghost removed the status/in-progress In progress label May 8, 2019
@alanshaw alanshaw deleted the feat/pull-mplex branch May 8, 2019 05:58
@alanshaw alanshaw mentioned this pull request May 13, 2019
44 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants