-
Notifications
You must be signed in to change notification settings - Fork 36
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 to Go 1.12 #583
Comments
I would wait until we update go-conventions: https://github.com/src-d/guide/blob/master/engineering/conventions/go.md |
I'm going to edit this issue to drop |
but then we need another issue for |
But we can open it when the conventions guidelines are updated, don't you think? |
I understood that it was a thing to do from a message from @smola on company Slack channels. |
Alright, then feel free to open a new issue for |
agree with Carlos on that. There is also a question about |
When I opened this issue I considered it would be like an umbrella for such different tasks and discussions... but ok, here is the new issue. |
Since Go 1.12 is already there and both
Go v1.11
andGo v1.12
supports Go Modules, we could also update.travis
tov1.12.x
and migrate fromdep
togo mod
The text was updated successfully, but these errors were encountered: