-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
Upgrade golang version? #1381
Comments
maybe we can add notes in README? |
Yes @appleboy and add notes when starting gin, it's my option. |
@thinkerou Could you send the PR? thanks. |
@appleboy Done, but now only add some readme and debug warn, when we upgrade go version later, we also need update these info. |
I'm on go 1.10.x but still getting this GIN-debug warning GIN-debug] [WARNING] Now Gin requires Go 1.6 or later and Go 1.7 will be required soon . why ??? |
@bjm88 because the warning document |
@thinkerou should the warning message not be printed if you are using a version higher then the required? I don't think I understand your comment about the warning being "fixed". Thanks! I'm using 1.11 at the moment and was confused when I saw the warning thinking gin was somehow using an older version. Other users might run into a similar issue. |
@bjm88 @battlebas Hi, on #1572 I add go version judge, please help me review, thanks! |
Suggestion:
|
I agree with others that this warning message about requiring Go 1.6 or later should not be output when we are running a newer version. I am on 1.12 and still see that warning every time a start a server that uses gin. |
@appleboy @thinkerou Looks like we need to move on this sooner rather than latter. It looks like all Somehow in the
|
Yes.
This is a daring plan. But I think we should only remove |
Sorry, We should only kill 1.6 & 1.7 until we get |
#1851 merged! |
Now golang have released version 1.10, and gin have not use 1.7 or 1.8
context
feature and so on.We may point out
require golang 1.7
andrequire golang 1.8
period of time.ref: https://golang.google.cn/doc/devel/release.html#policy
The text was updated successfully, but these errors were encountered: