-
Notifications
You must be signed in to change notification settings - Fork 165
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
make sure that it uses the latest patch release available #206
Conversation
cpanato
commented
Sep 30, 2022
- make sure that it uses the latest patch release available
Signed-off-by: cpanato <[email protected]>
@@ -35,7 +35,8 @@ jobs: | |||
- name: Setup Go | |||
uses: actions/setup-go@v3 | |||
with: | |||
go-version: 1.19.1 | |||
go-version: '1.19' | |||
check-latest: true |
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.
Isn't this being bumped by dependabot already when new versions are available? What happens here if there's a 1.20 available?
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.
dependabot does not bump this part, it only bumps the action itself
if 1.20 is out we need to update and push a PR
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.
Right, dependabot will only change that part. When 1.20 is out, will we get build failures, warnings, or things like that?
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.
no, we will still run go1.19 only if we upgrade to go1.20
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 clarification!
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 review
@@ -35,7 +35,8 @@ jobs: | |||
- name: Setup Go | |||
uses: actions/setup-go@v3 | |||
with: | |||
go-version: 1.19.1 | |||
go-version: '1.19' | |||
check-latest: true |
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 clarification!