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

🌱 Upgrade to go 1.20 #1262

Merged
merged 2 commits into from
Sep 13, 2023
Merged

Conversation

spencerschrock
Copy link
Member

One of our dependencies specifies 1.20 in its go.mod, so the upgrade is failing. (#1254). At some point we'll probably need to bump again to 1.21.

Note: our Dockerfile already uses 1.21, but ultimately the runtime behavior is based on whats in our go.mod

the docker version is specified as a tag. The comment will just get outdated.

Signed-off-by: Spencer Schrock <sschrock@google.com>
at the minimum this is needed to upgrade sigstore/cosign/v2.

Signed-off-by: Spencer Schrock <sschrock@google.com>
@codecov
Copy link

codecov bot commented Sep 13, 2023

Codecov Report

Merging #1262 (9706981) into main (0ebbf54) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1262   +/-   ##
=======================================
  Coverage   63.17%   63.17%           
=======================================
  Files           4        4           
  Lines         296      296           
=======================================
  Hits          187      187           
  Misses         94       94           
  Partials       15       15           

@spencerschrock spencerschrock merged commit e79dcb6 into ossf:main Sep 13, 2023
@spencerschrock spencerschrock deleted the bump-go-1.20 branch September 13, 2023 17:42
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.

2 participants