Skip to content

ci: pin release Go version to go.mod - #1250

Merged
madhavilosetty-intel merged 1 commit into
mainfrom
ci/pin-release-go-version
Sep 4, 2026
Merged

ci: pin release Go version to go.mod#1250
madhavilosetty-intel merged 1 commit into
mainfrom
ci/pin-release-go-version

Conversation

@madhavilosetty-intel

Copy link
Copy Markdown
Contributor

setup-go satisfied ">=1.23.0" with the runner's pre-installed Go 1.26.7 and GOTOOLCHAIN=local was in effect, so the release build failed against the go 1.27.0 directive in go.mod. Read the version from go.mod instead, as ci.yml already does, so a future bump of the go directive cannot desync the release job.

setup-go satisfied ">=1.23.0" with the runner's pre-installed Go 1.26.7
and GOTOOLCHAIN=local was in effect, so the release build failed against
the go 1.27.0 directive in go.mod. Read the version from go.mod instead,
as ci.yml already does, so a future bump of the go directive cannot
desync the release job.
@madhavilosetty-intel
madhavilosetty-intel requested a review from a team as a code owner September 4, 2026 16:57
@madhavilosetty-intel
madhavilosetty-intel enabled auto-merge (squash) September 4, 2026 16:58
@codecov

codecov Bot commented Sep 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.71%. Comparing base (df8cdd6) to head (38c8d96).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1250   +/-   ##
=======================================
  Coverage   56.71%   56.71%           
=======================================
  Files         149      149           
  Lines       12154    12154           
=======================================
  Hits         6893     6893           
  Misses       5260     5260           
  Partials        1        1           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@madhavilosetty-intel
madhavilosetty-intel merged commit 8156048 into main Sep 4, 2026
21 checks passed
@madhavilosetty-intel
madhavilosetty-intel deleted the ci/pin-release-go-version branch September 4, 2026 17:27
@RosieAMT

RosieAMT commented Sep 4, 2026

Copy link
Copy Markdown

🎉 This PR is included in version 1.40.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@rsdmike

rsdmike commented Sep 4, 2026

Copy link
Copy Markdown
Member

Would recommend to not do this, creates overhead and pins to a specific version in go.mod meaning more frequent updates to the go.mod file. Recommend to simply update constraint to 1.25.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants