Linter version update to v2.12.2

[petrpinkas]

No description provided.

[qodo-for-securesign]

PR Summary by Qodo

Pin golangci-lint GitHub Action to v9 and golangci-lint to v2.12.2

⚙️ Configuration changes 🕐 Less than 5 minutes

AI Description

• Bump golangci-lint GitHub Action from v7 to v9 in CI.
• Pin golangci-lint binary version to v2.12.2 instead of latest.
• Improve CI determinism by avoiding implicit lint version upgrades.

Diagram

graph TD
  A["GitHub push / PR"] --> B["CI workflow: linter.yml"] --> C["Action: golangci-lint-action@v9"] --> D["Tool: golangci-lint v2.12.2"] --> E["Lint results (check run)"]

Loading

High-Level Assessment

The following are alternative approaches to this PR:

1. Pin GitHub Action by commit SHA

• ➕ Best supply-chain security (immutable reference)
• ➕ Avoids unexpected behavior changes within a moving tag
• ➖ Less readable than semantic version tags
• ➖ Requires periodic manual SHA updates

2. Track golangci-lint `latest`

• ➕ Always picks up the newest linter fixes/rules automatically
• ➕ No need to manually bump linter versions
• ➖ Non-deterministic CI (sudden new failures)
• ➖ Harder to reproduce historical results locally

Recommendation: The PR’s approach (pinning golangci-lint to a specific version and updating the action major) is the best default for reproducible CI. If tighter supply-chain controls are a goal, consider additionally pinning golangci/golangci-lint-action to a commit SHA instead of a moving @v9 tag.

Files changed (1) +2 / -2

Other (1) +2 / -2

linter.ymlUpdate golangci-lint action to v9 and pin linter to v2.12.2 +2/-2

Update golangci-lint action to v9 and pin linter to v2.12.2

• Upgrades the GitHub Action used for linting from 'golangci-lint-action@v7' to '@v9'. Switches the linter binary from 'latest' to an explicit 'v2.12.2' to make CI runs deterministic.

.github/workflows/linter.yml

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 3.78%. Comparing base (1860749) to head (46e8957).

Additional details and impacted files

@@          Coverage Diff          @@
##            main     #99   +/-   ##
=====================================
  Coverage   3.78%   3.78%           
=====================================
  Files          9       9           
  Lines       2536    2536           
=====================================
  Hits          96      96           
  Misses      2433    2433           
  Partials       7       7           

Flag	Coverage Δ
unit	3.78% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

[qodo-for-securesign]

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review