Skip to content

fix golangci-lint version to match ci pipeline and precommit hook#234

Merged
dangra merged 2 commits into
mainfrom
lint-version-fix
May 6, 2026
Merged

fix golangci-lint version to match ci pipeline and precommit hook#234
dangra merged 2 commits into
mainfrom
lint-version-fix

Conversation

@dangra

@dangra dangra commented May 6, 2026

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings May 6, 2026 18:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Pins the golangci-lint tool version used in GitHub Actions to match the version already pinned in the pre-commit hook, improving consistency between local and CI lint results.

Changes:

  • Pin golangci-lint in .github/workflows/checks.yml to v2.11.3 (instead of latest).
  • Add cross-referencing comments in both configs to keep the versions synchronized.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
.pre-commit-config.yaml Adds a note to keep the pre-commit golangci-lint rev aligned with CI.
.github/workflows/checks.yml Pins the CI golangci-lint version and adds a note pointing back to pre-commit config.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/checks.yml Outdated
Comment thread .github/workflows/checks.yml Outdated
uses: golangci/golangci-lint-action@v9
with:
version: latest
# Keep golangci-lint version insync with .pre-commit-config.yml
Comment thread .pre-commit-config.yaml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@dangra dangra merged commit edb3b51 into main May 6, 2026
5 checks passed
@dangra dangra deleted the lint-version-fix branch May 6, 2026 18:45
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