Skip to content

Commit eaa62ab

Browse files
authored
chore(deps): Update dependency golangci/golangci-lint to v2 (#514)
This PR contains the following updates: | Package | Update | Change | Pending | |---|---|---|---| | [golangci/golangci-lint](https://redirect.github.com/golangci/golangci-lint) | major | `v1.64.8` -> `v2.1.6` | `v2.2.1` (+1) | --- ### Release Notes <details> <summary>golangci/golangci-lint (golangci/golangci-lint)</summary> ### [`v2.1.6`](https://redirect.github.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v216) [Compare Source](https://redirect.github.com/golangci/golangci-lint/compare/v2.1.5...v2.1.6) 1. Linters bug fixes - `godot`: from 1.5.0 to 1.5.1 - `musttag`: from 0.13.0 to 0.13.1 2. Documentation - Add note about golangci-lint v2 integration in VS Code ### [`v2.1.5`](https://redirect.github.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v215) [Compare Source](https://redirect.github.com/golangci/golangci-lint/compare/v2.1.4...v2.1.5) Due to an error related to Snapcraft, some artifacts of the v2.1.4 release have not been published. This release contains the same things as v2.1.3. ### [`v2.1.4`](https://redirect.github.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v214) [Compare Source](https://redirect.github.com/golangci/golangci-lint/compare/v2.1.3...v2.1.4) Due to an error related to Snapcraft, some artifacts of the v2.1.3 release have not been published. This release contains the same things as v2.1.3. ### [`v2.1.3`](https://redirect.github.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v213) [Compare Source](https://redirect.github.com/golangci/golangci-lint/compare/v2.1.2...v2.1.3) 1. Linters bug fixes - `fatcontext`: from 0.7.2 to 0.8.0 2. Misc. - migration: fix `nakedret.max-func-lines: 0` - migration: fix order of `staticcheck` settings - fix: add `go.mod` hash to the cache salt - fix: use diagnostic position for related information position ### [`v2.1.2`](https://redirect.github.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v212) [Compare Source](https://redirect.github.com/golangci/golangci-lint/compare/v2.1.1...v2.1.2) 1. Linters bug fixes - `exptostd`: from 0.4.2 to 0.4.3 - `gofumpt`: from 0.7.0 to 0.8.0 - `protogetter`: from 0.3.13 to 0.3.15 - `usetesting`: from 0.4.2 to 0.4.3 ### [`v2.1.1`](https://redirect.github.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v211) [Compare Source](https://redirect.github.com/golangci/golangci-lint/compare/v2.1.0...v2.1.1) The release process of v2.1.0 failed due to a regression inside goreleaser. The binaries of v2.1.0 have been published, but not the other artifacts (AUR, Docker, etc.). ### [`v2.1.0`](https://redirect.github.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v210) [Compare Source](https://redirect.github.com/golangci/golangci-lint/compare/v2.0.2...v2.1.0) 1. Enhancements - Add an option to display absolute paths (`--path-mode=abs`) - Add configuration path placeholder (`${config-path}`) - Add `warn-unused` option for `fmt` command - Colored diff for `fmt` command (`golangci-lint fmt --diff-colored`) 2. New linters - Add `funcorder` linter https://github.com/manuelarte/funcorder 3. Linters new features or changes - `go-errorlint`: from 1.7.1 to 1.8.0 (automatic error comparison and type assertion fixes) - ⚠️ `goconst`: `ignore-strings` is deprecated and replaced by `ignore-string-values` - `goconst`: from 1.7.1 to 1.8.1 (new options: `find-duplicates`, `eval-const-expressions`) - `govet`: add `httpmux` analyzer - `nilnesserr`: from 0.1.2 to 0.2.0 (detect more cases) - `paralleltest`: from 1.0.10 to 1.0.14 (checks only `_test.go` files) - `revive`: from 1.7.0 to 1.9.0 (support kebab case for setting names) - `sloglint`: from 0.9.0 to 0.11.0 (autofix, new option `msg-style`, suggest `slog.DiscardHandler`) - `wrapcheck`: from 2.10.0 to 2.11.0 (new option `report-internal-errors`) - `wsl`: from 4.6.0 to 4.7.0 (cgo files are always excluded) 4. Linters bug fixes - `fatcontext`: from 0.7.1 to 0.7.2 - `gocritic`: fix `importshadow` checker - `gosec`: from 2.22.2 to 2.22.3 - `ireturn`: from 0.3.1 to 0.4.0 - `loggercheck`: from 0.10.1 to 0.11.0 - `nakedret`: from 2.0.5 to 2.0.6 - `nonamedreturns`: from 1.0.5 to 1.0.6 - `protogetter`: from 0.3.12 to 0.3.13 - `testifylint`: from 1.6.0 to 1.6.1 - `unconvert`: update to HEAD 5. Misc. - Fixes memory leaks when using go1.(N) with golangci-lint built with go1.(N-X) - Adds `golangci-lint-fmt` pre-commit hook 6. Documentation - Improvements - Updates section about vscode integration ### [`v2.0.2`](https://redirect.github.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v202) [Compare Source](https://redirect.github.com/golangci/golangci-lint/compare/v2.0.1...v2.0.2) 1. Misc. - Fixes flags parsing for formatters - Fixes the filepath used by the exclusion `source` option 2. Documentation - Adds a section about flags migration - Cleaning pages with v1 options ### [`v2.0.1`](https://redirect.github.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v201) [Compare Source](https://redirect.github.com/golangci/golangci-lint/compare/v2.0.0...v2.0.1) 1. Linters/formatters bug fixes - `golines`: fix settings during linter load 2. Misc. - Validates the `version` field before the configuration - `forbidigo`: fix migration ### [`v2.0.0`](https://redirect.github.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v200) [Compare Source](https://redirect.github.com/golangci/golangci-lint/compare/v1.64.8...v2.0.0) 1. Enhancements - 🌟 New `golangci-lint fmt` command with dedicated [formatter configuration](https://golangci-lint.run/welcome/quick-start/#formatting) - ♻️ New `golangci-lint migrate` command to help migration from v1 to v2 (cf. [Migration guide](https://golangci-lint.run/product/migration-guide/#command-migrate)) - ⚠️ New default values (cf. [Migration guide](https://golangci-lint.run/product/migration-guide/)) - ⚠️ No exclusions by default (cf. [Migration guide](https://golangci-lint.run/product/migration-guide/#issuesexclude-use-default)) - ⚠️ New default sort order (cf. [Migration guide](https://golangci-lint.run/product/migration-guide/#outputsort-order)) - 🌟 New option `run.relative-path-mode` (cf. [Migration guide](https://golangci-lint.run/product/migration-guide/#runrelative-path-mode)) - 🌟 New linters configuration (cf. [Migration guide](https://golangci-lint.run/product/migration-guide/#linters)) - 🌟 New output format configuration (cf. [Migration guide](https://golangci-lint.run/product/migration-guide/#output)) - 🌟 New `--fast-only` flag (cf. [Migration guide](https://golangci-lint.run/product/migration-guide/#lintersfast)) - 🌟 New option `linters.exclusions.warn-unused` to log a warning if an exclusion rule is unused. 2. New linters/formatters - Add `golines` formatter https://github.com/segmentio/golines 3. Linters new features - ⚠️ Merge `staticcheck`, `stylecheck`, `gosimple` into one linter (`staticcheck`) (cf. [Migration guide](https://golangci-lint.run/product/migration-guide/#lintersenablestylecheckgosimplestaticcheck)) - `go-critic`: from 0.12.0 to 0.13.0 - `gomodguard`: from 1.3.5 to 1.4.1 (block explicit indirect dependencies) - `nilnil`: from 1.0.1 to 1.1.0 (new option: `only-two`) - `perfsprint`: from 0.8.2 to 0.9.1 (checker name in the diagnostic message) - `staticcheck`: new `quickfix` set of rules - `testifylint`: from 1.5.2 to 1.6.0 (new options: `equal-values`, `suite-method-signature`, `require-string-msg`) - `wsl`: from 4.5.0 to 4.6.0 (new option: `allow-cuddle-used-in-block`) 4. Linters bug fixes - `bidichk`: from 0.3.2 to 0.3.3 - `errchkjson`: from 0.4.0 to 0.4.1 - `errname`: from 1.0.0 to 1.1.0 - `funlen`: fix `ignore-comments` option - `gci`: from 0.13.5 to 0.13.6 - `gosmopolitan`: from 1.2.2 to 1.3.0 - `inamedparam`: from 0.1.3 to 0.2.0 - `intrange`: from 0.3.0 to 0.3.1 - `protogetter`: from 0.3.9 to 0.3.12 - `unparam`: from [`8a5130c`](https://redirect.github.com/golangci/golangci-lint/commit/8a5130ca722f) to [`0df0534`](https://redirect.github.com/golangci/golangci-lint/commit/0df0534333a4) 5. Misc. - 🧹 Configuration options renaming (cf. [Migration guide](https://golangci-lint.run/product/migration-guide/)) - 🧹 Remove options (cf. [Migration guide](https://golangci-lint.run/product/migration-guide/)) - 🧹 Remove flags (cf. [Migration guide](https://golangci-lint.run/product/migration-guide/)) - 🧹 Remove alternative names (cf. [Migration guide](https://golangci-lint.run/product/migration-guide/#alternative-linter-names)) - 🧹 Remove or replace deprecated elements (cf. [Migration guide](https://golangci-lint.run/product/migration-guide/)) - Adds an option to display some commands as JSON: - `golangci-lint config path --json` - `golangci-lint help linters --json` - `golangci-lint help formatters --json` - `golangci-lint linters --json` - `golangci-lint formatters --json` - `golangci-lint version --json` 6. Documentation - [Migration guide](https://golangci-lint.run/product/migration-guide/) </details> --- ### Configuration 📅 **Schedule**: Branch creation - Between 12:00 AM and 03:59 AM, on day 1 of the month ( * 0-3 1 * * ) (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. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://redirect.github.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0MC4yMi4xIiwidXBkYXRlZEluVmVyIjoiNDAuMjIuMSIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==-->
1 parent baa0e79 commit eaa62ab

File tree

2 files changed

+88
-79
lines changed

2 files changed

+88
-79
lines changed

.github/workflows/lint_golang.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ jobs:
2929
restore-keys: |
3030
${{runner.os}}-go-${{steps.go.outputs.go-version}}-lint
3131
- name: golangci-lint
32-
uses: golangci/golangci-lint-action@v6
32+
uses: golangci/golangci-lint-action@v7
3333
with:
34-
version: v1.64.8
34+
version: v2.1.6
3535
args: --verbose

.golangci.yml

Lines changed: 86 additions & 77 deletions
Original file line numberDiff line numberDiff line change
@@ -1,92 +1,101 @@
1+
version: "2"
12
run:
23
tests: true
3-
timeout: 10m
4-
issues:
5-
exclude-dirs:
6-
- internal/pb
7-
max-issues-per-linter: 0
8-
max-same-issues: 0
9-
10-
linters-settings:
11-
errcheck:
12-
check-blank: false
13-
gocritic:
14-
disabled-checks:
15-
- commentFormatting
16-
dupl:
17-
# tokens count to trigger issue, 150 by default
18-
threshold: 500
19-
decorder:
20-
dec-order:
21-
- type
22-
- const
23-
- var
24-
- func
25-
disable-dec-order-check: false
26-
govet:
27-
enable-all: true
28-
disable:
29-
- fieldalignment
30-
- shadow
31-
32-
revive:
33-
enable-all-rules: true
34-
rules:
35-
- name: cyclomatic
36-
disabled: true
37-
- name: argument-limit
38-
disabled: true
39-
- name: function-length
40-
disabled: true
41-
- name: function-result-limit
42-
disabled: true
43-
- name: line-length-limit
44-
disabled: true
45-
- name: file-header
46-
disabled: true
47-
- name: cognitive-complexity
48-
disabled: true
49-
- name: banned-characters
50-
disabled: true
51-
- name: max-public-structs
52-
disabled: true
53-
- name: add-constant
54-
disabled: true
55-
- name: unhandled-error
56-
disabled: true
57-
- name: deep-exit
58-
disabled: true
59-
- name: nested-structs
60-
disabled: true
61-
- name: import-alias-naming
62-
disabled: true
63-
- name: unchecked-type-assertion
64-
disabled: true
65-
66-
gofmt:
67-
rewrite-rules:
68-
- pattern: "interface{}"
69-
replacement: "any"
70-
- pattern: "a[b:len(a)]"
71-
replacement: "a[b:]"
72-
734
linters:
745
enable:
756
- asciicheck
767
- bodyclose
778
- dupl
78-
- errcheck
799
- gocritic
80-
- gofmt
81-
- gosimple
82-
- govet
83-
- ineffassign
8410
- importas
8511
- misspell
8612
- nakedret
8713
- prealloc
8814
- revive
89-
- staticcheck
9015
- unconvert
9116
- unparam
92-
- unused
17+
settings:
18+
decorder:
19+
dec-order:
20+
- type
21+
- const
22+
- var
23+
- func
24+
disable-dec-order-check: false
25+
dupl:
26+
threshold: 500
27+
errcheck:
28+
check-blank: false
29+
gocritic:
30+
disabled-checks:
31+
- commentFormatting
32+
govet:
33+
disable:
34+
- fieldalignment
35+
- shadow
36+
enable-all: true
37+
revive:
38+
enable-all-rules: true
39+
rules:
40+
- name: cyclomatic
41+
disabled: true
42+
- name: argument-limit
43+
disabled: true
44+
- name: function-length
45+
disabled: true
46+
- name: function-result-limit
47+
disabled: true
48+
- name: line-length-limit
49+
disabled: true
50+
- name: file-header
51+
disabled: true
52+
- name: cognitive-complexity
53+
disabled: true
54+
- name: banned-characters
55+
disabled: true
56+
- name: max-public-structs
57+
disabled: true
58+
- name: add-constant
59+
disabled: true
60+
- name: unhandled-error
61+
disabled: true
62+
- name: deep-exit
63+
disabled: true
64+
- name: nested-structs
65+
disabled: true
66+
- name: import-alias-naming
67+
disabled: true
68+
- name: unchecked-type-assertion
69+
disabled: true
70+
exclusions:
71+
generated: lax
72+
presets:
73+
- comments
74+
- common-false-positives
75+
- legacy
76+
- std-error-handling
77+
paths:
78+
- internal/pb
79+
- third_party$
80+
- builtin$
81+
- examples$
82+
issues:
83+
max-issues-per-linter: 0
84+
max-same-issues: 0
85+
formatters:
86+
enable:
87+
- gofmt
88+
settings:
89+
gofmt:
90+
rewrite-rules:
91+
- pattern: interface{}
92+
replacement: any
93+
- pattern: a[b:len(a)]
94+
replacement: a[b:]
95+
exclusions:
96+
generated: lax
97+
paths:
98+
- internal/pb
99+
- third_party$
100+
- builtin$
101+
- examples$

0 commit comments

Comments
 (0)