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

build: update Gin minimum Go version to 1.21 #3960

Merged
merged 2 commits into from May 9, 2024
Merged

Conversation

appleboy
Copy link
Member

@appleboy appleboy commented May 9, 2024

  • Update the minimum Go version requirement for Gin from 1.20 to 1.21 in both debug.go and debug_test.go
  • Modify the warning message to reflect the new minimum Go version requirement in debug.go
  • Adjust the test assertion to match the updated warning message in debug_test.go

since #3210 PR import quic-go/quic-go package which require go v1.21 above.

- Update the minimum Go version requirement for Gin from `1.20` to `1.21` in both `debug.go` and `debug_test.go`
- Modify the warning message to reflect the new minimum Go version requirement in `debug.go`
- Adjust the test assertion to match the updated warning message in `debug_test.go`

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>
@appleboy appleboy added this to the v1.11 milestone May 9, 2024
Copy link

codecov bot commented May 9, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 99.19%. Comparing base (3dc1cd6) to head (61dbaa6).
Report is 50 commits behind head on master.

Files Patch % Lines
debug.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3960      +/-   ##
==========================================
- Coverage   99.21%   99.19%   -0.03%     
==========================================
  Files          42       43       +1     
  Lines        3182     2722     -460     
==========================================
- Hits         3157     2700     -457     
+ Misses         17       12       -5     
- Partials        8       10       +2     
Flag Coverage Δ
?
-tags "sonic avx" 99.18% <0.00%> (?)
-tags go_json 99.18% <0.00%> (?)
-tags nomsgpack 99.17% <0.00%> (?)
go-1.18 ?
go-1.19 ?
go-1.20 ?
go-1.21 99.19% <0.00%> (-0.03%) ⬇️
go-1.22 99.19% <0.00%> (?)
macos-latest 99.17% <0.00%> (-0.04%) ⬇️
ubuntu-latest 99.19% <0.00%> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@appleboy appleboy requested a review from thinkerou May 9, 2024 03:15
@thinkerou
Copy link
Member

@thinkerou
Copy link
Member

- Update supported Go versions for GitHub actions to `1.21` and `1.22`
- Specify the required Go version as `1.21` or above in README
- Change code block syntax to `sh` in installation and demo run instructions
- Remove empty lines in README sections
- Update project list formatting without changing the content

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>
@appleboy
Copy link
Member Author

appleboy commented May 9, 2024

@thinkerou All done in 61dbaa6

@thinkerou thinkerou merged commit 7e29806 into gin-gonic:master May 9, 2024
23 of 25 checks passed
@appleboy appleboy deleted the go branch May 9, 2024 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants