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

structure refresh #89

Merged
merged 1 commit into from
Apr 9, 2016
Merged

structure refresh #89

merged 1 commit into from
Apr 9, 2016

Conversation

daviddias
Copy link
Member

Simple PR with structure refresh. Biggest contribution is updating the specs list with the specs that we have and that need to have.

Will continue working on separate PRs for the specs that still don't exist.



![](ipfs-splash.png)
> This repository contains the specs for the IPFS Protocol and associated subsystems. Some day we will hopefully transform these specs into RFCs. For now, they assume a high level of familiarity with the concepts.
Copy link
Member

Choose a reason for hiding this comment

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

The > section should match the repo description.

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated it to: This repository contains the specs for the IPFS Protocol and associated subsystems.. Sounds good?

Copy link
Member

Choose a reason for hiding this comment

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

Better to have this paragraph beneath, just not with a >. Put the > above it, the one you corrected it to.

@RichardLitt
Copy link
Member

Overall good. Some comments.

@@ -0,0 +1,102 @@
CLI - Command Line Interface
============================
Copy link
Member Author

Choose a reason for hiding this comment

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

@RichardLitt CLI stuff is here :)

@daviddias daviddias force-pushed the spec-refresh branch 4 times, most recently from 81e6ee9 to 21cf600 Compare April 1, 2016 21:54
@daviddias
Copy link
Member Author

Thank you @RichardLitt, made changes following your comments.

@jbenet
Copy link
Member

jbenet commented Apr 9, 2016

LGTM

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.

3 participants