XSAM / otelsql

OpenTelemetry instrumentation for database/sql
Apache License 2.0
277 stars 48 forks source link

Update module github.com/golangci/golangci-lint to v1.58.0 #251

Closed renovate[bot] closed 2 months ago

renovate[bot] commented 2 months ago

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
github.com/golangci/golangci-lint v1.52.2 -> v1.58.0 age adoption passing confidence

Release Notes

golangci/golangci-lint (github.com/golangci/golangci-lint) ### [`v1.58.0`](https://togithub.com/golangci/golangci-lint/compare/v1.57.2...v1.58.0) [Compare Source](https://togithub.com/golangci/golangci-lint/compare/v1.57.2...v1.58.0) ### [`v1.57.2`](https://togithub.com/golangci/golangci-lint/compare/v1.57.1...v1.57.2) [Compare Source](https://togithub.com/golangci/golangci-lint/compare/v1.57.1...v1.57.2) ### [`v1.57.1`](https://togithub.com/golangci/golangci-lint/releases/tag/v1.57.1) [Compare Source](https://togithub.com/golangci/golangci-lint/compare/v1.57.0...v1.57.1) #### Changelog - [`87b6bf1`](https://togithub.com/golangci/golangci-lint/commit/87b6bf17) build(deps): bump github.com/golangci/plugin-module-register from 0.1.0 to 0.1.1 ([#​4549](https://togithub.com/golangci/golangci-lint/issues/4549)) - [`921d535`](https://togithub.com/golangci/golangci-lint/commit/921d5357) build(deps): bump github.com/pelletier/go-toml/v2 from 2.1.1 to 2.2.0 ([#​4548](https://togithub.com/golangci/golangci-lint/issues/4548)) - [`cd890db`](https://togithub.com/golangci/golangci-lint/commit/cd890db2) fix: filter invalid issues before other processors ([#​4552](https://togithub.com/golangci/golangci-lint/issues/4552)) ### [`v1.57.0`](https://togithub.com/golangci/golangci-lint/compare/v1.56.2...v1.57.0) [Compare Source](https://togithub.com/golangci/golangci-lint/compare/v1.56.2...v1.57.0) ### [`v1.56.2`](https://togithub.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v1562) [Compare Source](https://togithub.com/golangci/golangci-lint/compare/v1.56.1...v1.56.2) 1. updated linters - `go-critic`: from 0.11.0 to 0.11.1 - `gosec`: from 2.18.2 to 2.19.0 - `testifylint`: from 1.1.1 to 1.1.2 - `usestdlibvars`: from 1.24.0 to 1.25.0 - `wsl`: from 4.2.0 to 4.2.1 2. misc. - Fix missing version in Docker image 3. Documentation - Explain the limitation of `new-from-rev` and `new-from-patch` ### [`v1.56.1`](https://togithub.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v1561) [Compare Source](https://togithub.com/golangci/golangci-lint/compare/v1.56.0...v1.56.1) 1. updated linters - `errcheck`: from 1.6.3 to 1.7.0 - `govet`: disable `loopclosure` with go1.22 - `revive`: from 1.3.6 to 1.3.7 - `testifylint`: from 1.1.0 to 1.1.1 ### [`v1.56.0`](https://togithub.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v1560) [Compare Source](https://togithub.com/golangci/golangci-lint/compare/v1.55.2...v1.56.0) 1. new linters - `spancheck`: https://github.com/jjti/go-spancheck 2. updated linters - `depguard`: from 2.1.0 to 2.2.0 - `exhaustive`: from 0.11.0 to 0.12.0 - `exhaustruct`: from 3.1.0 to 3.2.0 - `gci`: from 0.11.2 to 0.12.1 - `ginkgolinter`: from 0.14.1 to 0.15.2 - `go-check-sumtype`: from 0.1.3 to 0.1.4 - `go-critic`: from 0.9.0 to 0.11.0 - `go-errorlint`: from 1.4.5 to 1.4.8 - `go-spancheck`: from 0.4.2 to 0.5.2 - `goconst`: from 1.6.0 to 1.7.0 - `godot`: from 1.4.15 to 1.4.16 - `gofumpt`: from 0.5.0 to 0.6.0 - `inamedparam`: from 0.1.2 to 0.1.3 - `ineffassign`: from 0.0.0-20230610083614-0e73809eb601 to 0.1.0 - `ireturn`: from 0.2.2 to 0.3.0 - `misspell`: add mode option - `musttag`: from v0.7.2 to v0.8.0 - `paralleltest`: from 1.0.8 to 1.0.9 - `perfsprint`: from 0.2.0 to 0.6.0 - `protogetter`: from 0.2.3 to 0.3.4 - `revive`: from 1.3.4 to 1.3.6 - `sloglint`: add static-msg option - `sloglint`: from 0.1.2 to 0.4.0 - `testifylint`: from 0.2.3 to 1.1.0 - `unparam`: from [`2022122`](https://togithub.com/golangci/golangci-lint/commit/20221223090309)-7455f1af531d to [`2024010`](https://togithub.com/golangci/golangci-lint/commit/20240104100049)-c549a3470d14 - `whitespace`: update after moving to the `analysis` package - `wsl`: from 3.4.0 to 4.2.0 - `zerologlint`: from 0.1.3 to 0.1.5 3. misc. - πŸŽ‰ go1.22 support - Implement stats per linter with a flag - Make versioning inside Docker image consistent with binaries - Parse Go RC version 4. Documentation - Fix `noctx` description - Add missing fields to `.golangci.reference.yml` - Improve `.golangci.reference.yml` defaults - `typecheck`: improve FAQ - `exhaustruct`: note that struct regular expressions are expected to match the entire `package/name/structname` - `wrapcheck`: adjust `ignoreSigs` to new defaults **Important** `testifylint` has [breaking changes](https://togithub.com/Antonboom/testifylint/releases/tag/v1.0.0) about enabling/disabling checks: - If you were using the option `enable` with a filtered list of checks, you should either add `disable-all: true` (1) or use `disable` field (2). ```yml ``` ### Example (1) testifylint: disable-all: true enable: - bool-compare - compares - empty - error-is-as - error-nil - expected-actual - go-require - float-compare - len - nil-compare - require-error ### - suite-dont-use-pkg - suite-extra-assert-call - suite-thelper ``` ```yml ### Example (2) testifylint: disable: - suite-dont-use-pkg ``` ### [`v1.55.2`](https://togithub.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v1552) [Compare Source](https://togithub.com/golangci/golangci-lint/compare/v1.55.1...v1.55.2) 1. updated linters - `ireturn`: from 0.2.1 to 0.2.2 - `ginkgolinter`: from 0.14.0 to 0.14.1 ### [`v1.55.1`](https://togithub.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v1551) [Compare Source](https://togithub.com/golangci/golangci-lint/compare/v1.55.0...v1.55.1) 1. updated linters - `gosec`: from 2.18.1 to 2.18.2 2. misc. - `revgrep`: from v0.5.0 to v0.5.2 (support git < 2.41.0) - output: convert backslashes to forward slashes for GitHub Action annotations printer ### [`v1.55.0`](https://togithub.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v1550) [Compare Source](https://togithub.com/golangci/golangci-lint/compare/v1.54.2...v1.55.0) 1. new linters - `gochecksumtype`: https://github.com/alecthomas/go-check-sumtype - `inamedparam`: https://github.com/macabu/inamedparam - `perfsprint`: https://github.com/catenacyber/perfsprint - `protogetter`: https://github.com/ghostiam/protogetter - `sloglint`: https://github.com/go-simpler/sloglint - `testifylint`: https://github.com/Antonboom/testifylint 2. updated linters - `bidichk`: from 0.2.4 to 0.2.7 - `decorder`: from 0.4.0 to 0.4.1 - `dupword`: from 0.0.12 to 0.0.13 - `errchkjson`: from 0.3.1 to 0.3.6 - `gci`: from 0.11.0 to 0.11.2 - `ginkgolinter`: from 0.13.5 to 0.14.0 - `go-errorlint`: from 1.4.4 to 1.4.5 - `gocognit`: from 1.0.7 to 1.1.0 - `goconst`: from 1.5.1 to 1.6.0 - `godot`: from 1.4.14 to 1.4.15 - `gofmt`: update to HEAD - `goimports`: update to HEAD - `gosec`: from 2.17.0 to 2.18.1 - `gosmopolitan`: from 1.2.1 to 1.2.2 - `govet`: add `appends` analyzer - `ireturn`: from 0.2.0 to 0.2.1 - `protogetter`: from 0.2.2 to 0.2.3 - `revgrep`: from [`745bb2f`](https://togithub.com/golangci/golangci-lint/commit/745bb2f7c2e6) to v0.5.0 - `revive`: from 1.3.2 to 1.3.4 - `sqlclosecheck`: from 0.4.0 to 0.5.1 - `staticcheck`: from 0.4.5 to 0.4.6 - `tagalign`: from 1.3.2 to 1.3.3 - `unused`: support passing in options 3. misc. - Add a pre-commit hook to check all files 4. Documentation - add source options to exclude-rules docs - `gosec`: add G602 to includes/excludes inside .golangci.reference.yml ### [`v1.54.2`](https://togithub.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v1542) [Compare Source](https://togithub.com/golangci/golangci-lint/compare/v1.54.1...v1.54.2) 1. updated linters: - `errname`: from 0.1.10 to 0.1.12 - `ginkgolinter`: from 0.13.3 to 0.13.5 - `go-errorlint`: from 1.4.3 to 1.4.4 - `godot`: from 1.4.11 to 1.4.14 - `gosec`: from 2.16.0 to 2.17.0 - `musttag`: from 0.7.1 to 0.7.2 - `nilnil`: from 0.1.5 to 0.1.7 - `staticcheck`: from 0.4.3 to 0.4.5 - `usestdlibvars`: from 1.23.0 to 1.24.0 - `govet`: add missing `directive` and `slog` passes ### [`v1.54.1`](https://togithub.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v1541) [Compare Source](https://togithub.com/golangci/golangci-lint/compare/v1.54.0...v1.54.1) 1. updated linters: - `go-critic`: from 0.8.2 to 0.9.0 2. misc. - plugin: temporarily hide warning about using plugins using the old API ### [`v1.54.0`](https://togithub.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v1540) [Compare Source](https://togithub.com/golangci/golangci-lint/compare/v1.53.3...v1.54.0) 1. updated linters: - `decorder`: from 0.2.3 to 0.4.0 - `dupword`: from 0.0.11 to 0.0.12 - `errorlint`: from 1.4.2 to 1.4.3 - `exhaustruct`: from 2.3.0 to 3.1.0 - `forbidigo`: from 1.5.3 to 1.6.0 - `funlen`: from 0.0.3 to 0.1.0 - `gci`: from 0.10.1 to 0.11.0 - `ginkgolinter`: from 0.12.1 to 0.13.3 - `go-critic`: from 0.8.1 to 0.8.2 - `go-errorlint`: from 1.4.2 to 1.4.3 - `go-exhaustruct`: from 2.3.0 to 3.1.0 - `gocognit`: from 1.0.6 to 1.0.7 - `gocritic`: from 0.8.1 to 0.8.2 - `gofmt`: autofix missing newline at EOF - `misspell`: 0.4.0 to 0.4.1 - `musttag`: from 0.7.0 to 0.7.1 - `paralleltest`: from 1.0.7 to 1.0.8 - `tagalign`: from 1.2.2 to 1.3.2 - `typecheck`: explain it and remove it from the linter list - `zerologlint`: from 0.1.2 to 0.1.3 2. misc. - πŸŽ‰ go1.21 support - plugin: include custom linters in `enable-all` - plugin: allow to use settings for plugins 3. Documentation - Add linter descriptions. **Important** `ruleguard` (a "rule" inside `gocritic`) was disabled in this release (v1.54.0) and was enabled again in the next release (v1.54.1). `exhaustruct` has breaking changes with regular expressions, more details [here](https://togithub.com/GaijinEntertainment/go-exhaustruct/releases/tag/v3.0.0). ### [`v1.53.3`](https://togithub.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v1533) [Compare Source](https://togithub.com/golangci/golangci-lint/compare/v1.53.2...v1.53.3) 1. updated linters: - `cyclop`: from 1.2.0 to 1.2.1 - `exhaustive`: from 0.10.0 to 0.11.0 - `forbidigo`: from 1.5.1 to 1.5.3 - `ginkgolinter`: from 0.12.2 to 0.12.1 - `ineffassign`: bump to HEAD - `nakedret`: from 2.0.1 to 2.0.2 - `zerologlint`: from 0.1.1 to 0.1.2 2. misc. - codeclimate: reduce memory allocation - support illumos/amd64 ### [`v1.53.2`](https://togithub.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v1532) [Compare Source](https://togithub.com/golangci/golangci-lint/compare/v1.53.1...v1.53.2) 1. updated linters - `depguard`: from v2.0.1 to 2.1.0 2. misc. - `depguard`: throw error only when the linter is called ### [`v1.53.1`](https://togithub.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v1531) [Compare Source](https://togithub.com/golangci/golangci-lint/compare/v1.53.0...v1.53.1) 1. misc. - `depguard`: fix GOROOT detection - `importas`: fix duplication detection when aliases use regular expression replacement pattern ### [`v1.53.0`](https://togithub.com/golangci/golangci-lint/blob/HEAD/CHANGELOG.md#v1530) [Compare Source](https://togithub.com/golangci/golangci-lint/compare/v1.52.2...v1.53.0) 1. new linters - `gosmopolitan`: https://github.com/xen0n/gosmopolitan - `mirror`: https://github.com/butuzov/mirror - `tagalign`: https://github.com/4meepo/tagalign - `zerologlint`: https://github.com/ykadowak/zerologlint 2. updated linters - `bodyclose`: bump to HEAD - `containedctx`: from 1.0.2 to 1.0.3 - `depguard`: migrate to v2 - `errname`: from 0.1.9 to 0.1.10 - `exhaustive`: from 0.9.5 to 0.10.0 - `forbidigo`: better support for configuring complex rules - `gci`: improve error message - `ginkgolinter`: add suppress-async-assertion option - `ginkgolinter`: from 0.9.0 to 0.12.0 - `go-critic`: from 0.7.0 to 0.8.1 - `go-errorlint`: from 1.4.0 to 1.4.2 - `gofumpt`: from 0.4.0 to 0.5.0 - `gosec`: convert global settings as map with proper key type - `gosec`: from 2.15.0 to 2.16.0 - `importas`: detect duplicate alias or package in the configuration - `importas`: fix typo in logs - `ireturn`: from 0.1.1 to 0.2.0 - `musttag`: from 0.5.0 to 0.7.0 - `nakedret`: to 2.0.1 - `nilnil`: from 0.1.3 to 0.1.5 - `paralleltest`: from 1.0.6 to 1.0.7 - `revive`: from 1.3.1 to 1.3.2 - `tagliatelle`: from 0.4.0 to 0.5.0 - `usestdlibvars`: fix configuration 3. misc. - `golang.org/x/tools`: from 0.7.0 to 0.9.2 - add loongarch64 to the install script - output: add colored-tab - remove warning when the config file is explicitly stdin - rules: support inverted path match 4. Documentation - `mnd`: clarify ignore usage examples to be regexps - `tagliatelle`: update documentation - improve features section - update supported Go versions FAQ

Configuration

πŸ“… Schedule: Branch creation - At any time (no schedule defined), 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.



This PR has been generated by Mend Renovate. View repository job log here.

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.1%. Comparing base (9a21e25) to head (85294a4).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #251 +/- ## ===================================== Coverage 84.1% 84.1% ===================================== Files 13 13 Lines 605 605 ===================================== Hits 509 509 Misses 72 72 Partials 24 24 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

renovate[bot] commented 2 months ago

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠ Warning: custom changes will be lost.