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

🌱 Migrate to go 1.21 #3387

Merged
merged 3 commits into from
Aug 14, 2023
Merged

🌱 Migrate to go 1.21 #3387

merged 3 commits into from
Aug 14, 2023

Conversation

spencerschrock
Copy link
Member

What kind of change does this PR introduce?

dependency update

What is the current behavior?

  • use go1.19, which is no longer supported

What is the new behavior (if this is a feature change)?**

  • use go1.21 for our CI and builds/releases

  • keep go1.19 in our go.mod as we don't currently use any 1.20 or 1.21 features. this is for our dependents.

  • Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes

#3386

Special notes for your reviewer

Does this PR introduce a user-facing change?

For user-facing changes, please add a concise, human-readable release note to
the release-note

(In particular, describe what changes users might need to make in their
application as a result of this pull request.)

Scorecard releases will now use go1.21

Signed-off-by: Spencer Schrock <sschrock@google.com>
Signed-off-by: Spencer Schrock <sschrock@google.com>
…v variable.

Signed-off-by: Spencer Schrock <sschrock@google.com>
@spencerschrock spencerschrock temporarily deployed to gitlab August 14, 2023 17:53 — with GitHub Actions Inactive
@spencerschrock spencerschrock temporarily deployed to integration-test August 14, 2023 17:53 — with GitHub Actions Inactive
@codecov
Copy link

codecov bot commented Aug 14, 2023

Codecov Report

Merging #3387 (9b7c9ae) into main (198aa96) will decrease coverage by 6.22%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3387      +/-   ##
==========================================
- Coverage   74.30%   68.08%   -6.22%     
==========================================
  Files         178      175       -3     
  Lines       12876    12856      -20     
==========================================
- Hits         9567     8753     -814     
- Misses       2766     3593     +827     
+ Partials      543      510      -33     

Copy link
Member

@justaugustus justaugustus left a comment

Choose a reason for hiding this comment

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

Thanks @spencerschrock!
(Filed a separate issue for the code coverage hit: #3388)

@justaugustus justaugustus merged commit 5ec66fa into ossf:main Aug 14, 2023
40 of 41 checks passed
@spencerschrock spencerschrock deleted the dep/go1.21 branch August 14, 2023 19:24
ashearin pushed a commit to kgangerlm/scorecard-gitlab that referenced this pull request Nov 13, 2023
* Bump dockerfiles to 1.21
* Go minimum version should match our go.mod
* Bump GitHub action go version to 1.21 and ensure all workflows use env variable.

---------

Signed-off-by: Spencer Schrock <sschrock@google.com>
Signed-off-by: Allen Shearin <allen.p.shearin@gmail.com>
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