Thanks to visit codestin.com
Credit goes to github.com

Skip to content

add nargs linter #5753

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
2 of 3 tasks
cospectrum opened this issue Apr 21, 2025 · 2 comments
Closed
2 of 3 tasks

add nargs linter #5753

cospectrum opened this issue Apr 21, 2025 · 2 comments
Labels
declined

Comments

@cospectrum
Copy link

cospectrum commented Apr 21, 2025 •
edited
Loading

Welcome

How did you install golangci-lint?

Nix

Your feature request related to a problem? Please describe

This issue has been closed and forgotten by bot: #553

Describe the solution you'd like

Add nargs linter support (should be disabled by default).

Describe alternatives you've considered

unparam is not enough for some use cases.

Additional context

No response

Supporter
@cospectrum cospectrum added the enhancement New feature or improvement label Apr 21, 2025
@boring-cyborg boring-cyborg
Copy link

boring-cyborg bot commented Apr 21, 2025

Hey, thank you for opening your first Issue ! 🙂 If you would like to contribute we have a guide for contributors.

@ldez ldez added declined and removed enhancement New feature or improvement labels Apr 21, 2025
@ldez
Copy link
Member

ldez commented Apr 21, 2025

Hello,

From the narg readme:

Unlike the unparamlinter, this linter is aggressive by design and may have false positives

Based on our rules, this analyzer is a detector, and not a linter, because of the number of false positives.
So it will not be added to golangci-lint.

But you can use it as a plugin: https://golangci-lint.run/plugins/module-plugins/

@ldez ldez closed this as completed Apr 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
declined
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ldez @cospectrum