Skip to content

Commit 055f704

Browse files
authored
chore(deps): Update dependency golangci/golangci-lint to v2 (#644)
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.5` | `v2.1.6` | --- ### Release Notes <details> <summary>golangci/golangci-lint (golangci/golangci-lint)</summary> ### [`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:eyJjcmVhdGVkSW5WZXIiOiIzOS4yNjQuMCIsInVwZGF0ZWRJblZlciI6IjM5LjI2NC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->
1 parent 1209a4f commit 055f704

File tree

2 files changed

+86
-77
lines changed

2 files changed

+86
-77
lines changed

.github/workflows/lint_golang.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ jobs:
1717
with:
1818
go-version-file: go.mod
1919
- name: golangci-lint
20-
uses: golangci/golangci-lint-action@v6
20+
uses: golangci/golangci-lint-action@v7
2121
with:
22-
version: v1.64.8
22+
version: v2.1.5
2323
args: --verbose

.golangci.yml

Lines changed: 84 additions & 75 deletions
Original file line numberDiff line numberDiff line change
@@ -1,90 +1,99 @@
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-
linters-settings:
10-
errcheck:
11-
check-blank: false
12-
gocritic:
13-
disabled-checks:
14-
- commentFormatting
15-
dupl:
16-
# tokens count to trigger issue, 150 by default
17-
threshold: 500
18-
decorder:
19-
dec-order:
20-
- type
21-
- const
22-
- var
23-
- func
24-
disable-dec-order-check: false
25-
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: unchecked-type-assertion
62-
disabled: true
63-
64-
gofmt:
65-
rewrite-rules:
66-
- pattern: "interface{}"
67-
replacement: "any"
68-
- pattern: "a[b:len(a)]"
69-
replacement: "a[b:]"
70-
714
linters:
725
enable:
736
- asciicheck
747
- bodyclose
758
- dupl
76-
- errcheck
779
- gocritic
78-
- gofmt
79-
- gosimple
80-
- govet
81-
- ineffassign
8210
- importas
8311
- misspell
8412
- nakedret
8513
- prealloc
8614
- revive
87-
- staticcheck
8815
- unconvert
8916
- unparam
90-
- 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: unchecked-type-assertion
67+
disabled: true
68+
exclusions:
69+
generated: lax
70+
presets:
71+
- comments
72+
- common-false-positives
73+
- legacy
74+
- std-error-handling
75+
paths:
76+
- internal/pb
77+
- third_party$
78+
- builtin$
79+
- examples$
80+
issues:
81+
max-issues-per-linter: 0
82+
max-same-issues: 0
83+
formatters:
84+
enable:
85+
- gofmt
86+
settings:
87+
gofmt:
88+
rewrite-rules:
89+
- pattern: interface{}
90+
replacement: any
91+
- pattern: a[b:len(a)]
92+
replacement: a[b:]
93+
exclusions:
94+
generated: lax
95+
paths:
96+
- internal/pb
97+
- third_party$
98+
- builtin$
99+
- examples$

0 commit comments

Comments
 (0)