Skip to content
This repository was archived by the owner on Apr 16, 2020. It is now read-only.

Standardized Readme #66

Merged
merged 3 commits into from
Jul 29, 2016
Merged

Standardized Readme #66

merged 3 commits into from
Jul 29, 2016

Conversation

RichardLitt
Copy link
Contributor


[![](https://img.shields.io/badge/made%20by-Protocol%20Labs-blue.svg?style=flat-square)](http://ipn.io)
[![](https://img.shields.io/badge/project-IPFS-blue.svg?style=flat-square)](http://ipfs.io/)
[![](https://img.shields.io/badge/freenode-%23ipfs-blue.svg?style=flat-square)](http://webchat.freenode.net/?channels=%23ipfs)
Copy link
Collaborator

Choose a reason for hiding this comment

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

same comment as ipfs/apps#28 ;)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Haha. Let's talk about it there.

@JesseWeinstein
Copy link

LGTM; why did you decide this needed a separate LICENSE file, but ipfs/apps didn't?

@RichardLitt
Copy link
Contributor Author

I didn't add one to ipfs/apps because it was one of the first PRs I did, and I failed to add one to my own oversight.

Changed License to point out that this is not a code repo. Should be good to go; one more pass after that last commit, anyone?

@JesseWeinstein
Copy link

Still LGTM.

@davidar davidar merged commit f0a488d into master Jul 29, 2016
@davidar davidar deleted the feature/standardize-readme branch July 29, 2016 08:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants