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

chore(go): Add support go 1.20 #3484

Merged
merged 4 commits into from
Feb 12, 2023
Merged

Conversation

mstmdev
Copy link
Contributor

@mstmdev mstmdev commented Feb 2, 2023

No description provided.

@codecov
Copy link

codecov bot commented Feb 2, 2023

Codecov Report

Merging #3484 (891cdd3) into master (e02ae6a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3484   +/-   ##
=======================================
  Coverage   98.63%   98.63%           
=======================================
  Files          42       42           
  Lines        3145     3145           
=======================================
  Hits         3102     3102           
  Misses         29       29           
  Partials       14       14           
Flag Coverage Δ
98.63% <ø> (ø)
go-1.16 ∅ <ø> (∅)
go-1.17 98.53% <ø> (ø)
go-1.18 98.53% <ø> (ø)
go-1.19 98.63% <ø> (ø)
go-1.20 98.63% <ø> (?)
macos-latest 98.53% <ø> (ø)
ubuntu-latest 98.63% <ø> (ø)

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

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@@ -31,7 +31,7 @@ jobs:
strategy:
matrix:
os: [ubuntu-latest, macos-latest]
go: [1.16, 1.17, 1.18, 1.19]
go: ['1.16', '1.17', '1.18', '1.19', '1.20']
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why add '?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

It does not appear to be correctly identified as go1.20 here.
@thinkerou

@mstmdev
Copy link
Contributor Author

mstmdev commented Feb 2, 2023

Wait for sonic to support go1.20.
bytedance/sonic#355

@appleboy appleboy added enhancement dependencies Pull requests that update a dependency file labels Feb 5, 2023
@appleboy appleboy added this to the v1.9 milestone Feb 5, 2023
@mstmdev
Copy link
Contributor Author

mstmdev commented Feb 10, 2023

It's done, please review.
@thinkerou

@appleboy
Copy link
Member

@thinkerou Please help to review.

@thinkerou thinkerou merged commit bd82c9e into gin-gonic:master Feb 12, 2023
@mstmdev mstmdev deleted the support-go1.20 branch February 12, 2023 05:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants