Skip to content

Align lint workflow Go toolchain with go.mod requirement - #236

Draft
mdelapenya with Copilot wants to merge 2 commits into
dependabot/go_modules/go.opentelemetry.io/otel/sdk-1.43.0from
copilot/fix-lint-job-failure
Draft

Align lint workflow Go toolchain with go.mod requirement#236
mdelapenya with Copilot wants to merge 2 commits into
dependabot/go_modules/go.opentelemetry.io/otel/sdk-1.43.0from
copilot/fix-lint-job-failure

Conversation

Copilot AI commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

The lint GitHub Actions job failed because it resolved to a preinstalled Go toolchain older than the version required by go.mod. This change makes the lint workflow derive its Go version from the module declaration so lint runs against the same toolchain the project requires.

  • Root cause addressed

    • actions/setup-go in the lint workflow did not specify a version source, so CI selected go1.24.x while the module requires go1.25.0.
  • Workflow change

    • Updated .github/workflows/golangci-lint.yml to pin toolchain resolution to go.mod via go-version-file.
  • Resulting behavior

    • The lint job now uses the repository-declared Go version instead of a runner default, preventing version-mismatch failures during package loading.
- uses: actions/setup-go@v7
  with:
    go-version-file: 'go.mod'

Co-authored-by: mdelapenya <951580+mdelapenya@users.noreply.github.com>
Copilot AI changed the title Fix lint workflow Go version selection Align lint workflow Go toolchain with go.mod requirement Aug 11, 2026
Copilot AI requested a review from mdelapenya August 11, 2026 16:21
@mdelapenya

Copy link
Copy Markdown
Member

@copilot there are conflicts, resolve them

Co-authored-by: mdelapenya <951580+mdelapenya@users.noreply.github.com>

Copilot AI commented Aug 11, 2026

Copy link
Copy Markdown
Contributor Author

@copilot there are conflicts, resolve them

Resolved by updating the lint workflow to align with the current main-branch action version while keeping the Go toolchain fix. Addressed in commit 798c278.

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.

2 participants