-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
go version should be at least 1.16 because of io.Discard
(Since Go 1.16, Feb 2021)…
#16764
Conversation
…, so go version should be at least 1.16 now.
Codecov Report
@@ Coverage Diff @@
## main #16764 +/- ##
==========================================
+ Coverage 45.34% 45.36% +0.01%
==========================================
Files 760 760
Lines 85504 85504
==========================================
+ Hits 38774 38785 +11
+ Misses 40439 40432 -7
+ Partials 6291 6287 -4
Continue to review full report at Codecov.
|
@wxiaoguang Discard do work with 1.14 any points in bumping requirement? |
@6543 we've already defined 1.16 as minimum, so it is good to have it in the places defined in this PR |
oh yes that one dep. |
The release notes for Gitea 1.15.0 say that Go 1.15 or later is required, but this PR made the requirement 1.16 or later. |
DAMN we should have updated the blog at that point. Realistically we can't support 1.15 as upstream projects like golang-jwt/jwt will not support out-of-support golang versions - and will not even attempt to provide security fixes for them. I agree its not great but as 1.15 was released after the end of life for go 1.15 I don't think we can say we can support 1.15. (If you really need to use go 1.15 the source for Gitea code will compile with go 1.15 if you change the Makefile.) |
I've put up a PR to update/fix the blog https://gitea.com/gitea/blog/pulls/171 |
I'm good, I upgraded to Go 1.17 and Gitea 1.15 is working great (and quite a bit faster than 1.14 was when built with Go 1.15). |
This is no accident, I (and lunny) have sent substantial numbers of PRs to improve speed. The one most likely responsible for your experience improvement is #16059. |
Currently the go version is
1.14
ingo.mod
, actually there are many usage ofio.Discard
(Since Go 1.16, Feb 2021) in codeSo go version should be at least 1.16 now. And
README.md
is also updated.