Skip to content

Conversation

@maxday
Copy link
Member

@maxday maxday commented May 31, 2025

Context

This PR fixes the first todo of #1838
It uses the documented go-version-file instead of an hardcoded value (doc: https://github.com/actions/setup-go?tab=readme-ov-file#getting-go-version-from-the-gomod-file)

Even if the go version is printed out in the output of actions/setup-go I think it's useful to print it in the build target (see Makefile) for potential debugging purpose, totally fine to remove it if needed.

Test

Tested on my fork ✅

  1. Output of actions/setup-go
Screenshot 2025-05-31 at 10 02 14 AM
  1. Output of the go version log in make build
Screenshot 2025-05-31 at 10 07 10 AM

@maxday maxday requested a review from a team as a code owner May 31, 2025 09:09
@tylerbenson tylerbenson merged commit d9668be into open-telemetry:main Jun 3, 2025
14 checks passed
@tylerbenson tylerbenson added the github_actions Pull requests that update GitHub Actions code label Jun 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants