Skip to content

Take the CI Go version from go.mod - #46

Merged
umputun merged 1 commit into
masterfrom
ci/go-version-from-go-mod
Nov 6, 2026
Merged

umputun merged 1 commit into
masterfrom
ci/go-version-from-go-mod

Conversation

@paskal

@paskal paskal commented Nov 1, 2026

Copy link
Copy Markdown
Collaborator

Previously, the workflow pinned the Go version in actions/setup-go separately from go.mod. After this change, CI reads the major and minor version from the go directive in go.mod and installs the latest patch release of it with check-latest: true. go-version-file would install exactly the patch version written in go.mod, such as 1.25.0, so it is not used. go.mod is unchanged.

The resolved version does not change: CI installs the latest Go 1.25 patch release, as before.

On this branch, setup-go installs Go 1.25.14 and the build passes.

CI pinned its Go version separately from go.mod. It now reads the major
and minor version from the go directive and installs the latest patch
release of it.
@paskal
paskal requested a review from umputun as a code owner November 1, 2026 10:45
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 41298869691

Coverage decreased (-0.3%) to 95.993%

Details

  • Coverage decreased (-0.3%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 2 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

2 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
telegram.go 2 95.67%

Coverage Stats

Coverage Status
Relevant Lines: 599
Covered Lines: 575
Line Coverage: 95.99%
Coverage Strength: 188.54 hits per line

💛 - Coveralls

@umputun
umputun merged commit 4487069 into master Nov 6, 2026
5 checks passed
@paskal
paskal deleted the ci/go-version-from-go-mod branch November 7, 2026 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants