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

[trivial] typo in README #40

Merged
merged 1 commit into from Oct 16, 2016
Merged

Conversation

jnewbery
Copy link
Contributor

Fix a trivial typo in README

@carryforward
Copy link

this left me scratching my head too.

Copy link
Member

@Roasbeef Roasbeef left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! I'd totally missed this typo previously, and can see how it can possibly be very confusing for developers attempting to build the project. When my two minor nits are addressed, I'll get this merged swiftly.

@@ -15,7 +15,7 @@ Don't try to port it to mainnet or an altcoin and use it today! No really. Lig
* **Go 1.5 or 1.6**

Installation instructions can be found here: http://golang.org/doc/install. It is recommended to add `$GOPATH/bin` to your `PATH` at this point.
**Note:** If you are building with Go 1.5, then you'll need to enable the vendor experiment by setting the `GO15VENDOREXPERIMENT` environment variable to `1`. If you're using Go 1.5, then it is safe to skip this step.
**Note:** If you are building with Go 1.5, then you'll need to enable the vendor experiment by setting the `GO15VENDOREXPERIMENT` environment variable to `1`. If you're using Go 1.6, then it is safe to skip this step.
Copy link
Member

Choose a reason for hiding this comment

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

Minor nit, this should read:

If you're using Go 1.6 or later...

@@ -15,7 +15,7 @@ Don't try to port it to mainnet or an altcoin and use it today! No really. Lig
* **Go 1.5 or 1.6**
Copy link
Member

Choose a reason for hiding this comment

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

While you're at it, can you modify this header to read Go 1.5+? Thanks!

@jnewbery
Copy link
Contributor Author

nit fixed

@Roasbeef Roasbeef merged commit 72ed6bf into lightningnetwork:master Oct 16, 2016
@Roasbeef
Copy link
Member

Thanks!

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.

None yet

3 participants