-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
bump to golang 1.16 #39
base: master
Are you sure you want to change the base?
Conversation
closes tj#36 closes datacharmer/dbdeployer#132 Just copying off d62cc32
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Any reason this is still not merged? |
Can someone merge the Dockerfile change? It doesn't look to me that it would be a biggie and I also encountered projects failing the compilation step due to the usage of 1.15.x. :-/ |
🐌 |
Same here! |
Kinda "forked" the same into my own tiny project, if someone would like to use that instead, supports 1.16 and I'm testing 1.17 builds |
@tj any news? |
closes #36
closes datacharmer/dbdeployer#132
Just copying off d62cc32
Please open an issue and discuss changes before spending time on them, unless the change is trivial or an issue already exists.