Skip to content

chore: downgrade golangci-lint version#119

Merged
dbrent-rp merged 2 commits into
mainfrom
PLAT-4740-downgrade-golangci-lint
May 2, 2026
Merged

chore: downgrade golangci-lint version#119
dbrent-rp merged 2 commits into
mainfrom
PLAT-4740-downgrade-golangci-lint

Conversation

@dbrent-rp
Copy link
Copy Markdown
Contributor

Description

Downgrading the glangci-lint version from 2.12.0 (released today) to 2.11.4
Jira Issue: https://revolutionparts.atlassian.net/browse/

Background

2.12.0 is far more aggressive with linting than 2.11.4 and the lint failures are blocking a P0 fix from going out

Testing Information

@dbrent-rp dbrent-rp requested a review from a team as a code owner May 2, 2026 00:55
@dbrent-rp dbrent-rp requested a review from aspencer May 2, 2026 00:55
@cursor
Copy link
Copy Markdown

cursor Bot commented May 2, 2026

PR Summary

Low Risk
Low risk: this only changes the CI lint tool version to reduce/avoid new lint failures; it does not affect runtime code or production behavior.

Overview
Pins the golangci-lint-action in .github/workflows/go-test.yaml to use golangci-lint v2.11.4 instead of latest, stabilizing CI linting behavior and avoiding unexpected failures from newer releases.

Reviewed by Cursor Bugbot for commit d483137. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 77c9585. Configure here.

Comment thread .github/workflows/go-test.yaml Outdated
@dbrent-rp dbrent-rp merged commit 17101f3 into main May 2, 2026
2 checks passed
@dbrent-rp dbrent-rp deleted the PLAT-4740-downgrade-golangci-lint branch May 2, 2026 01:12
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