-
Notifications
You must be signed in to change notification settings - Fork 38
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
Proposal: Graphsync Public Interfaces #20
Conversation
awesome docsing 👍 👍 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a formatting issue, otherwise this is amazing!
@hannahhoward the title of the PR says Graphsync Public Interfaces, but I feel that the interface may have been drowned out by the |
@lanzafame I agree there's a lot of "oh and here's some go-ipld-prime stuff you need to understand to make this work". I will probably separate that out into a separate compatibility doc at some point. But for now I'm inclined to present it together since it's relevant to all the early clients of the library just to get up and running kind of. |
1a1fc2a
to
dc54cee
Compare
Eek... I messed up something. Closing for a few minutes in case anyone is reading. |
Ok, sorry, problem solved. Change to initialization steps, FYI. |
Outline interfaces for using Graphsync in a readme.
Co-Authored-By: hannahhoward <[email protected]>
b81547f
to
dbfeeb1
Compare
Goals
Outline public interfaces for using GraphSync for review by Filecoin stakeholders
I did this by completing an initial Readme documenting project usage so that once approved we'll also have documentation. Recommend choosing the rich diff view when reviewing this PR