-
Notifications
You must be signed in to change notification settings - Fork 4k
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
THRIFT-5649: add github action for go #2697
Conversation
the cross test failure is irrelevant to the changes in this pr |
|
||
- uses: actions/setup-go@v3 | ||
with: | ||
go-version: '>=1.17.0' |
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.
the current supported go versions are 1.18.x and 1.19.x: https://github.com/apache/thrift/blob/master/LANGUAGES.md, not 1.17.x.
but looks like it actually picks the latest version (1.19.1) from the logs, so I think we probably actually need two actions, one for lib-go-1.18
with <1.19.0
and one for lib-go-1.19
with >=1.19.0
, or something 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.
thanks for the suggestion - will setup a new jira to follow up
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.
add github action for go
[skip ci]
anywhere in the commit message to free up build resources.