Skip to content

chore(deps): update golangci/golangci-lint-action action to v8#499

Closed
renovate[bot] wants to merge 1 commit intomainfrom
renovate/major-github-actions
Closed

chore(deps): update golangci/golangci-lint-action action to v8#499
renovate[bot] wants to merge 1 commit intomainfrom
renovate/major-github-actions

Conversation

@renovate
Copy link
Copy Markdown
Contributor

@renovate renovate Bot commented Jun 3, 2025

This PR contains the following updates:

Package Type Update Change
golangci/golangci-lint-action action major v6.5.2 -> v8.0.0

Release Notes

golangci/golangci-lint-action (golangci/golangci-lint-action)

v8.0.0

Compare Source

Requires golangci-lint version >= v2.1.0

What's Changed

Changes

Full Changelog: golangci/golangci-lint-action@v7...v8.0.0

v7.0.1

Compare Source

What's Changed

Documentation
Dependencies

New Contributors

Full Changelog: golangci/golangci-lint-action@v7.0.0...v7.0.1

v7.0.0

Compare Source

⚠️ The GitHub Action v7 supports golangci-lint v2 only. ⚠️

What's Changed

Changes
Documentation
Dependencies

New Contributors

Full Changelog: golangci/golangci-lint-action@v6.5.2...v7.0.0


Configuration

📅 Schedule: Branch creation - "before 8am every weekday" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate renovate Bot added the dependencies Pull requests that update a dependency file label Jun 3, 2025
@renovate renovate Bot requested review from a team as code owners June 3, 2025 06:26
@renovate renovate Bot added the dependencies Pull requests that update a dependency file label Jun 3, 2025
run: (echo -n "version="; make linter-version) >> "$GITHUB_OUTPUT"
- name: golangci-lint
uses: golangci/golangci-lint-action@55c2c1448f86e01eaae002a5a3a9624417608d84 # v6
uses: golangci/golangci-lint-action@4afd733a84b1f43292c63897423277bb7f4313a9 # v8.0.0
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This conflicts with our Makefile:

GOLANGCI_LINT_VERSION = "v1.64.5"

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.

Automatically updating golangci-lint is possibly not a good idea, as new version may introduce new linters with positives, which require manual fixes.
I'd suggest to use make lint in the CI and update the linter version in the Makefile every now and then (e.g. with every new minor Go version).

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.

That would also fix the "out-of-sync" situation with CI and Makefile.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

How does disabling updates on golangci-lint by renovate sound to you?

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.

That's OK as a quick fix.
Maybe in a separate PR: Add a CI step calling make lint

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Follow up with #506.

@renovate renovate Bot force-pushed the renovate/major-github-actions branch from 7e09ffd to 2a61748 Compare June 3, 2025 06:54
florianl added a commit that referenced this pull request Jun 4, 2025
Follow up to #499 (comment).
As automatic updates of the CI action golangci-lint-action conflicts with other parts, ignore this dependency.

Also set `automerge` for groups that will not break things.

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
@florianl
Copy link
Copy Markdown
Member

florianl commented Jun 5, 2025

Closing this PR in favor of #506.

@florianl florianl closed this Jun 5, 2025
@renovate renovate Bot deleted the renovate/major-github-actions branch June 5, 2025 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants