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

fix(actions): make sure to use latest Go version #270

Merged
merged 2 commits into from
Oct 7, 2022
Merged

Conversation

wass3r
Copy link
Collaborator

@wass3r wass3r commented Oct 6, 2022

waiting for actions/go-versions#60 to get merged to pull in 1.19.2 instead of 1.19.1 because that's what check_latest looks at to get the newest version.

I validated that this is need by rerunning https://github.com/go-vela/types/actions/runs/3171645351/jobs/5232434827 after the fix above was merged. It still pulled in the cached 1.19.1 version of Go.

We will want to apply this to all repos :octocat: (EDIT: ✅ PRs have been created)

@wass3r wass3r requested a review from a team as a code owner October 6, 2022 16:20
@codecov
Copy link

codecov bot commented Oct 6, 2022

Codecov Report

Merging #270 (ad4b377) into main (548cae4) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #270   +/-   ##
=======================================
  Coverage   96.93%   96.93%           
=======================================
  Files          57       57           
  Lines        6301     6301           
=======================================
  Hits         6108     6108           
  Misses        143      143           
  Partials       50       50           

@wass3r wass3r marked this pull request as draft October 6, 2022 16:27
@wass3r wass3r marked this pull request as ready for review October 7, 2022 13:23
@wass3r
Copy link
Collaborator Author

wass3r commented Oct 7, 2022

confirmed that this is now pulling in the newest version: https://github.com/go-vela/types/actions/runs/3205048265/jobs/5237076830#step:3:11

Copy link
Contributor

@jbrockopp jbrockopp left a comment

Choose a reason for hiding this comment

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

LGTM

@plyr4 plyr4 merged commit 9661c89 into main Oct 7, 2022
@plyr4 plyr4 deleted the fix/ci-go-install branch October 7, 2022 14:46
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.

3 participants