Skip to content

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 <[email protected]>
@appleboy appleboy added this to the v1.11 milestone May 9, 2024
@codecov
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 <[email protected]>
@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
@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.

2 participants