mirror of
https://github.com/jesseduffield/lazygit.git
synced 2025-05-11 04:15:48 +02:00
Add copyloopvar
to enabled linters (#3586)
Resolves #3573 It looks like the update to go1.22 is done by https://github.com/jesseduffield/lazygit/pull/3574. So it should be OK to close #3573 when this PR is merged. ## PR Description Add [`copyloopvar`](https://github.com/karamaru-alpha/copyloopvar) to enabled linters to detect unnecessary assignments in `for` loops. ## Context @jesseduffield and I were talking about necessity of this linter here. https://github.com/jesseduffield/lazygit/issues/3573#issuecomment-2123732829
This commit is contained in:
commit
ce3fe37787
1 changed files with 6 additions and 0 deletions
|
@ -14,10 +14,16 @@ linters:
|
||||||
- exhaustive
|
- exhaustive
|
||||||
- makezero
|
- makezero
|
||||||
- nakedret
|
- nakedret
|
||||||
|
- copyloopvar
|
||||||
# - goconst # TODO: enable and fix issues
|
# - goconst # TODO: enable and fix issues
|
||||||
fast: false
|
fast: false
|
||||||
|
|
||||||
linters-settings:
|
linters-settings:
|
||||||
|
copyloopvar:
|
||||||
|
# Check all assigning the loop variable to another variable.
|
||||||
|
# Default: false
|
||||||
|
# If true, an assignment like `a := x` will be detected as an error.
|
||||||
|
check-alias: true
|
||||||
exhaustive:
|
exhaustive:
|
||||||
default-signifies-exhaustive: true
|
default-signifies-exhaustive: true
|
||||||
staticcheck:
|
staticcheck:
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue