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

Go: Bump minimum version to 1.20 #12506

Merged
merged 10 commits into from
Nov 5, 2023
Merged

Conversation

tomponline
Copy link
Member

@tomponline tomponline commented Nov 5, 2023

This is the highest version we can go to at the moment due to readthedocs.
But that is actually fine as it allows us to update our dependencies to get latest security fixes (ala grpc), and still maintain compatibility with more Go versions.

Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
@tomponline tomponline self-assigned this Nov 5, 2023
@tomponline tomponline force-pushed the tp-go1.20 branch 5 times, most recently from 5a43a00 to 02dd4f5 Compare November 5, 2023 10:08
simondeziel and others added 7 commits November 5, 2023 12:09
Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
rand.Seed has been deprecated since Go 1.20

Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Otherwise it logs a line for every non-Go file!

Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
@tomponline tomponline changed the title Go: Bump minumum version to 1.20 Go: Bump minimum version to 1.20 Nov 5, 2023
roosterfish
roosterfish previously approved these changes Nov 5, 2023
Copy link
Contributor

@roosterfish roosterfish left a comment

Choose a reason for hiding this comment

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

Looks good to me!

x509.ParseCRL() has been deprecated since Go 1.19

Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
pkix.TBSCertificateList has been deprecated since Go 1.19 and replaced
by x509.RevocationList.

Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
@tomponline tomponline merged commit f14fc05 into canonical:main Nov 5, 2023
25 of 26 checks passed
@tomponline tomponline deleted the tp-go1.20 branch November 5, 2023 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants