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

docs: Make various README.md files consistent. #530

Merged
merged 1 commit into from Oct 23, 2015

Conversation

davecgh
Copy link
Member

@davecgh davecgh commented Oct 23, 2015

First, it removes the documentation section from all the README.md files and instead puts a web-based godoc badge and link at the top with the other badges. This is being done since the local godoc tool no longer ships with Go by default, so the instructions no longer work without first installing godoc. Due to this, pretty much everyone uses the web-based godoc these days anyways. Anyone who has manually installed godoc won't need instructions.

Second, it makes sure the ISC license badge is at the top with the other badges and removes the textual reference in the overview section.

Finally, it's modifies the Installation section to Installation and Updating and adds a -u to the go get command since it works for both and thus is simpler.

Review on Reviewable

@jcvernaleo
Copy link
Member

While I'm not happy about what has happened to godoc (I prefer it to the web stuff) that's the way it is going so we need to adapt.
This looks good to me.
OK

First, it removes the documentation section from all the README.md files
and instead puts a web-based godoc badge and link at the top with the
other badges.  This is being done since the local godoc tool no longer
ships with Go by default, so the instructions no longer work without
first installing godoc. Due to this, pretty much everyone uses the
web-based godoc these days anyways.  Anyone who has manually installed
godoc won't need instructions.

Second, it makes sure the ISC license badge is at the top with the other
badges and removes the textual reference in the overview section.

Finally, it's modifies the Installation section to Installation and
Updating and adds a '-u' to the 'go get' command since it works for both
and thus is simpler.
@conformal-deploy conformal-deploy merged commit 3942a11 into btcsuite:master Oct 23, 2015
@davecgh davecgh deleted the consistentreadme branch October 23, 2015 19:55
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.

None yet

3 participants