Skip to content

Conversation

@mjgarton
Copy link
Contributor

@mjgarton mjgarton commented Oct 27, 2025

mvdan/unparam@0df0534...5beb8c8


EDIT
Note from golangci-lint maintainer:

For future contributors: please don't open a PR to update a linter in these cases.
You should wait or open an issue.

Upgrade unparam dependency to fix a crasher bug

See mvdan/unparam#89
@CLAassistant
Copy link

CLAassistant commented Oct 27, 2025

CLA assistant check
All committers have signed the CLA.

@boring-cyborg
Copy link

boring-cyborg bot commented Oct 27, 2025

Hey, thank you for opening your first Pull Request !

@ldez ldez closed this Oct 27, 2025
@ldez ldez added the declined label Oct 27, 2025
@ldez
Copy link
Member

ldez commented Oct 27, 2025

This is a special dependency because there is no version, so dependabot is not able to update it automatically.

I regularly check those non-versioned dependencies, and I handle them.

For future contributors: please don't open a PR to update a linter in these cases.
You should wait or open an issue.

Exceptionally, I will reopen and merge this PR.

@ldez ldez reopened this Oct 27, 2025
@ldez ldez added dependencies Relates to an upstream dependency linter: update version Update version of linter go Pull requests that update Go code and removed declined labels Oct 27, 2025
@ldez ldez changed the title Upgrade unparam dependency build(deps): bump mvdan.cc/unparam to HEAD Oct 27, 2025
@ldez ldez added this to the unreleased milestone Oct 27, 2025
@ldez ldez self-requested a review October 27, 2025 18:47
@ldez ldez merged commit 4d2276d into golangci:main Oct 27, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Relates to an upstream dependency go Pull requests that update Go code linter: update version Update version of linter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants