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

Remove note about Go1.6 not being supported #2724

Merged
merged 1 commit into from
May 19, 2016

Conversation

Kubuxu
Copy link
Member

@Kubuxu Kubuxu commented May 17, 2016

Recommend Go1.6.2
Note that Go1.5 is still supported

License: MIT
Signed-off-by: Jakub Sztandera [email protected]

@Kubuxu Kubuxu added need/review Needs a review topic/docs-ipfs Topic docs-ipfs labels May 17, 2016
@@ -57,7 +57,8 @@ From there:

#### Install Go

First, you'll need Go. If you don't have it: [Download Go 1.5.2+](https://golang.org/dl/). **Go 1.6 is not yet supported.**
First, you'll need Go. If you don't have it: [Download Go 1.6.2+](https://golang.org/dl/). _Go 1.5 is still being supported._
Copy link
Member

Choose a reason for hiding this comment

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

I'd phrase it something like:

Ipfs requires at least go 1.5 to build, but we strongly recommend using 1.6.2. If you don't have it....

Copy link
Member Author

@Kubuxu Kubuxu May 18, 2016

Choose a reason for hiding this comment

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

I will phrase it as:
Build process of ipfs requires at least Go 1.5, but we strongly recommend using 1.6.2. If you don't have it....

Just to not capitalize ipfs (small habit from cjdns).

Recommend Go1.6.2
Note that Go1.5 is still supported

License: MIT
Signed-off-by: Jakub Sztandera <[email protected]>
@whyrusleeping
Copy link
Member

LGTM

@whyrusleeping whyrusleeping merged commit 0f58ceb into master May 19, 2016
@whyrusleeping whyrusleeping deleted the feature/Go1.6IsSupported branch May 19, 2016 02:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need/review Needs a review topic/docs-ipfs Topic docs-ipfs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants