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

update github.com/hashicorp/consul/api to v1.4.0 #569

Closed
wants to merge 1 commit into from
Closed

update github.com/hashicorp/consul/api to v1.4.0 #569

wants to merge 1 commit into from

Conversation

andyroyle
Copy link
Contributor

No description provided.

@aaronhurt
Copy link
Member

Could you update the go modules with the proper semantic versioning tag and ensure it matches the vendor in sources?

See here for more information with working with the new go modules system: https://github.com/golang/go/wiki/Modules

@andyroyle
Copy link
Contributor Author

My first time using go modules, sorry

@aaronhurt
Copy link
Member

@andyroyle this is also needed for #517 and #566 ... do you have the time to update it to use v1.4.0 as a module or would you like me to close it?

aaronhurt pushed a commit to myENA/fabio that referenced this pull request Nov 15, 2018
@magiconair magiconair changed the title vendoring in version 63aa189 of github.com/hashicorp/consul/api update github.com/hashicorp/consul/api to v1.4.0 Nov 15, 2018
@magiconair
Copy link
Contributor

LGTM

@aaronhurt aaronhurt closed this in 92c6115 Nov 15, 2018
@andyroyle
Copy link
Contributor Author

Cheers!

@aaronhurt
Copy link
Member

@andyroyle I hope I didn't step on your toes. There were a few other PRs waiting on an update of the library. Thank you very much for the contribution ... and the motivation to get it updated :)

@andyroyle
Copy link
Contributor Author

Not at all, I just wanted the updated consul client 👍

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.

3 participants