This repository has been archived by the owner on Feb 12, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Guide for review #15
Comments
@diasdavid Thanks for writing this up! Good to know there's no real PRs to review, just general overall QAQC. Will see what I can do. I don't understand pretty much anything of what is said in ipfs/specs#19, but I'm thinking of starting a document for technical overview and discussion to help out a) me b) future people like me. I'll add reviewing swarm components to my log of things to do. Sounds good. Thanks. |
MicrowaveDev
pushed a commit
to galtproject/js-ipfs
that referenced
this issue
May 22, 2020
They moved to `@hapi` namespace and now spew warnings on npm install. refs ipfs#2225 License: MIT Signed-off-by: Alan Shaw <alan.shaw@protocol.ai>
MicrowaveDev
pushed a commit
to galtproject/js-ipfs
that referenced
this issue
May 22, 2020
* Remove support for Node.js streams and pull streams * Adds a `urlSource` utility that will be exported by `js-ipfs-http-client` and `js-ipfs` along side `globSource` BREAKING CHANGE: Support for Node.js streams and Pull Streams has been removed
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
@RichardLitt, thank you so much for helping me review stuff, as I explain in IRC, there isn't exactly PR to review, since I'm pretty much the only one maintaining the code and I just keep iterating over master till we hit a alpha version.
There is a simple 'roadmap' in this repo's Readme, however, if it is the first time you are dialing in to libp2p, it might not be clear what is going on. For that, I've a WIP of the libp2p spec, it does explain a lot, unfortunately not all, yet. You can find it here - ipfs/specs#19
In terms of reviewing code, the best place to start would be by the Swarm components, which handles dialing, protocol muxing, stream muxing and Identify. It is what every other component uses so that they can open streams between peers.
The text was updated successfully, but these errors were encountered: