-
Notifications
You must be signed in to change notification settings - Fork 12
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
SANDBOX-677 use go-version-file #49
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great idea . lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR
@@ -19,16 +19,16 @@ jobs: | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you remove GO_VERSION: from env: too? at line 11 , it is not used anymore i guess
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
whoops, good eye, I somehow missed this - thought i had removed it.
removed in 6950e0f
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #49 +/- ##
=======================================
Coverage 67.03% 67.03%
=======================================
Files 43 43
Lines 2372 2372
=======================================
Hits 1590 1590
Misses 596 596
Partials 186 186 |
uses go-version-file instead of go-version. One less place to change when making updates.
Changes order in github action to first checkout code and then install go.
related PRs:
codeready-toolchain/api#434
codeready-toolchain/toolchain-common#415
codeready-toolchain/member-operator#584
codeready-toolchain/registration-service#445
codeready-toolchain/host-operator#1060
codeready-toolchain/toolchain-e2e#1014