elink21 commented on PR #23814: URL: https://github.com/apache/beam/pull/23814#issuecomment-1291022641
> It looks like we could either have the action always use the latest version of Go for building the boot scripts. This is recommended. The performance implications are minimal as go doesn't release minor or patch versions that often, and we already have downloading go as part of our gradle builds. https://github.com/actions/setup-go#check-latest-version > Getting the version from the Go Mod file is handy too but, since the boot scripts don't change often, and are part of the release, it's better for them to have been built with the most secure version of the standard libraries. https://github.com/actions/setup-go#getting-go-version-from-the-gomod-file > > I think both of these are new to the v3 version of the action however, so if INFRA poses a blocker on using v3 (they shouldn't) the previous recommendation applies. Many thanks for the recommendation @lostluck , I've tested with Go Version 1.19 and check-latest-version option set, the workflow is working correctly, we can go with latest version. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
