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

clientv3: add quote that Go v1.9 is required #8583

Closed
wants to merge 2 commits into from
Closed

clientv3: add quote that Go v1.9 is required #8583

wants to merge 2 commits into from

Conversation

ake-persson
Copy link

@ake-persson ake-persson commented Sep 20, 2017

Updated clientv3 with quote that Go v1.9 is required

#8548

@@ -10,6 +10,8 @@
go get github.com/coreos/etcd/clientv3
```

> Requires Go v1.9
Copy link
Contributor

Choose a reason for hiding this comment

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

probably make this bold instead?

@@ -10,7 +10,7 @@
go get github.com/coreos/etcd/clientv3
```

> Requires Go v1.9
> **Requires Go v1.9+**
Copy link
Contributor

Choose a reason for hiding this comment

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

remove > ?

Copy link
Contributor

Choose a reason for hiding this comment

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

it makes this less significant.

Copy link
Author

Choose a reason for hiding this comment

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

A quote is the closest thing to a "notice" in markdown. I think it draws more attention since it's a diff. color. But if you want it without no probs.

Copy link
Contributor

Choose a reason for hiding this comment

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

@gyuho what do you think?

Copy link
Member

Choose a reason for hiding this comment

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

do we need go 1.9 to build both etcd and etcd client? if that's the case, should we doc requirement on the front page?

Copy link
Contributor

Choose a reason for hiding this comment

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

Then maybe just update https://github.com/coreos/etcd#getting-etcd?

For those wanting to try the very latest version,

To

For those wanting to try the very latest version/client,

?

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

@fanminshi Yeah that needs to be fixed as well, in a separate patch.

Copy link
Member

Choose a reason for hiding this comment

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

@gyuho @xiang90 I prefer updating the front page about build requirement instead doing here. Less place to track.

Copy link
Contributor

Choose a reason for hiding this comment

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

@gyuho @fanminshi

clientv3 is usually used as a standalone library. it worth adding a link here to the go version requirement if we want to avoid duplication.

@ake-persson
Copy link
Author

Won't fix

@namgivu
Copy link

namgivu commented Oct 24, 2017

I try etcd v3.2.9 today with golang 1.9 and still getting the error

vendor/github.com/coreos/etcd/clientv3/client.go:336:9: undefined: metadata.NewContext
vendor/github.com/coreos/etcd/clientv3/lease.go:326:14: undefined: metadata.FromContext

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants