Skip to content

[codex] align Go version references#2524

Open
ianalloway wants to merge 1 commit into
IceWhaleTech:mainfrom
ianalloway:codex/codecov-go-version
Open

[codex] align Go version references#2524
ianalloway wants to merge 1 commit into
IceWhaleTech:mainfrom
ianalloway:codex/codecov-go-version

Conversation

@ianalloway

@ianalloway ianalloway commented Jun 27, 2026

Copy link
Copy Markdown

Summary

  • make the Codecov workflow read its Go version from go.mod
  • update the development prerequisites to require Go 1.21 or newer
  • pin the Codecov upload action to the full SHA for its v3 tag

Why

go.mod declares go 1.21, but the coverage workflow was pinned to Go 1.20 and the setup docs still mentioned Go > v1.17.0. Reading from go.mod keeps CI aligned when the module version changes later. The Codecov action is pinned to satisfy the workflow security quality gate.

Validation

  • checked the workflow now uses go-version-file: go.mod
  • checked the development docs now reference Go 1.21
  • verified codecov/codecov-action@v3 resolves to ab904c41d6ece82784817410c45d8b8c02684457
  • go test ./pkg/utils ./pkg/sqlite

Copilot AI review requested due to automatic review settings June 27, 2026 16:54

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@ianalloway ianalloway force-pushed the codex/codecov-go-version branch from 7418d62 to b10b4dc Compare June 27, 2026 16:56
@ianalloway ianalloway changed the title [codex] align coverage workflow Go version [codex] align Go version references Jun 27, 2026
@ianalloway ianalloway force-pushed the codex/codecov-go-version branch 2 times, most recently from a96c062 to cb7801c Compare June 27, 2026 17:05
Comment thread .github/workflows/codecov.yml Fixed
@ianalloway ianalloway force-pushed the codex/codecov-go-version branch from cb7801c to 5072f29 Compare June 27, 2026 17:08
@sonarqubecloud

Copy link
Copy Markdown

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