-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
fix: error message when trying to migrate a migrated config #5836
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.
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The check was done in the wrong place, so it was not triggered when the config was already migrated. Another error message was reported about jsonschema validation issues. Because a v2 config file does not validate against the v1 schema. That was misleading.
ldez
approved these changes
May 27, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
alaudaa-renovate bot
added a commit
to AlaudaDevops/golangci-lint
that referenced
this pull request
Aug 5, 2025
* build(deps): bump github.com/go-critic/go-critic from 0.12.0 to 0.13.0 (golangci#5579) * build(deps): bump mvdan.cc/unparam to HEAD (golangci#5584) * build(deps): bump github.com/timakin/bodyclose from ed6a65f985e3 to 1db5c5ca4d67 (golangci#5585) * docs: improve Docker example with cache reuse (golangci#5586) * docs: add a migration guide for VSCode integration (golangci#5587) * build(deps): bump github.com/ckaznocha/intrange from 0.3.0 to 0.3.1 (golangci#5589) * feat: add option stdin for fmt command (golangci#5588) * build(deps): bump github.com/daixiang0/gci from 0.13.5 to 0.13.6 (golangci#5592) * fix: funlen ignore-comments (golangci#5594) * docs: cleaning * chore: prepare release * docs: update GitHub Action assets (golangci#5595) * docs: fix displaying GoLand logo on a dark theme (golangci#5598) * fix: validate version before configuration (golangci#5599) * fix: copy golines settings during linter settings load (golangci#5607) * fix: forbidigo migration (golangci#5606) * docs: add example of the field version (golangci#5609) * chore: prepare release * docs: update GitHub Action assets (golangci#5611) Co-authored-by: Fernandez Ludovic <[email protected]> * docs: adaptive stargazers image (golangci#5613) * fix: enable formatters with flags (golangci#5620) * docs: add section about flags migration (golangci#5621) * docs: replace 'disable-all' with 'default: none' (golangci#5622) * docs: update reference comment for errchkjson (golangci#5623) * fix: formatter validation message (golangci#5624) * docs: info about the logo * docs: fix flags example * docs: fix flags example * docs: clean old configuration examples (golangci#5626) * fix: use absolute filepath inside base rule source (golangci#5629) * chore: prepare release * docs: update GitHub Action assets (golangci#5634) * docs: add version field to configuration sample (golangci#5632) * docs: remove golint in nolint example (golangci#5635) * build(deps): bump github.com/kunwardeep/paralleltest from 1.0.10 to 1.0.13 (golangci#5636) * docs: fix config reference for formatters (golangci#5638) * docs: improve version documentation (golangci#5639) * docs: fix settings examples (golangci#5643) * build(deps): bump tar-fs and puppeteer in /docs (golangci#5653) * build(deps): bump github.com/ghostiam/protogetter from 0.3.12 to 0.3.13 (golangci#5658) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump github.com/Antonboom/testifylint from 1.6.0 to 1.6.1 (golangci#5654) Co-authored-by: Fernandez Ludovic <[email protected]> * docs: cleanup config reference (golangci#5659) * dev: rewrite and simplify thanks page generation (golangci#5662) * docs: remove tenv linter settings (golangci#5665) * docs: improve default exclusions render (golangci#5661) * docs: fix `go get -tool` command (golangci#5669) * fix: gocritic importshadow checker (golangci#5673) * build(deps): bump the linter-testdata group across 2 directories with 4 updates (golangci#5676) Co-authored-by: Fernandez Ludovic <[email protected]> * docs: update demo (golangci#5677) * build(deps): bump github.com/alexkohler/nakedret/v2 from 2.0.5 to 2.0.6 (golangci#5681) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump github.com/shirou/gopsutil/v4 from 4.25.2 to 4.25.3 (golangci#5680) Co-authored-by: Fernandez Ludovic <[email protected]> * docs: add link to issues related to some integrations (golangci#5682) * build(deps): bump github.com/Crocmagnon/fatcontext from 0.7.1 to 0.7.2 (golangci#5685) * feat: add config path placeholder (golangci#5650) * feat: add an option to display absolute paths (golangci#5651) * feat: colored diff for fmt command (golangci#5652) * build(deps): bump github.com/tomarrell/wrapcheck/v2 from 2.10.0 to 2.11.0 (golangci#5656) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump github.com/kunwardeep/paralleltest from 1.0.13 to 1.0.14 (golangci#5657) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump github.com/mgechev/revive from 1.7.0 to 1.8.0 (golangci#5663) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump github.com/polyfloyd/go-errorlint from 1.7.1 to 1.8.0 (golangci#5686) * feat: add warn-unused option for fmt command (golangci#5668) * build(deps): bump github.com/firefart/nonamedreturns from 1.0.5 to 1.0.6 (golangci#5687) * build(deps): bump github.com/bombsimon/wsl/v4 from 4.6.0 to 4.7.0 (golangci#5689) * build(deps): bump github.com/alecthomas/chroma/v2 from 2.15.0 to 2.16.0 (golangci#5690) * build(deps): bump go-simpler.org/sloglint from 0.9.0 to 0.10.0 (golangci#5688) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump dependencies in the linter-testdata group (golangci#5691) Co-authored-by: Fernandez Ludovic <[email protected]> * docs: add `redundant-test-main-exit` to `revive` rules (golangci#5692) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump github.com/alingse/nilnesserr from 0.1.2 to 0.2.0 (golangci#5693) * build(deps): bump github.com/securego/gosec/v2 from 2.22.2 to 2.22.3 (golangci#5694) Co-authored-by: Fernandez Ludovic <[email protected]> * fix: memory leaks when using go1.(N) with golangci-lint built with with go1.(N-1) (golangci#5695) * build(deps): bump go-simpler.org/sloglint from 0.10.0 to 0.10.1 (golangci#5696) * docs: explicitly describe that the `migrate` command automatically migrate `linters.presets` (golangci#5697) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump golang.org/x/sys from 0.31.0 to 0.32.0 (golangci#5699) * build(deps): bump go-simpler.org/sloglint from 0.10.1 to 0.11.0 (golangci#5698) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump golang.org/x/oauth2 from 0.28.0 to 0.29.0 in /scripts/gen_github_action_config in the scripts group (golangci#5704) * feat: add golangci-lint-fmt pre-commit hook (golangci#5705) * build(deps): bump golang.org/x/tools from 0.31.0 to 0.32.0 (golangci#5708) * build(deps): bump github.com/butuzov/ireturn from 0.3.1 to 0.4.0 (golangci#5710) * build(deps): bump github.com/pelletier/go-toml/v2 from 2.2.3 to 2.2.4 (golangci#5711) * docs: update section about vscode integration (golangci#5706) Co-authored-by: logica0419 <[email protected]> * build(deps): bump github.com/jgautheron/goconst from 1.7.1 to 1.8.1 (golangci#5712) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump github.com/golangci/unconvert to HEAD (golangci#5713) * build(deps): bump github.com/timonwong/loggercheck from 0.10.1 to 0.11.0 (golangci#5715) * govet: add `httpmux` analyzer (golangci#5717) * dev: update JSON schema * build(deps): bump github.com/mgechev/revive from 1.8.0 to 1.9.0 (golangci#5721) * Add funcorder linter (golangci#5630) * chore: prepare release * docs: update GitHub Action assets (golangci#5723) * docs: update documentation assets (golangci#5722) Co-authored-by: Fernandez Ludovic <[email protected]> * chore: downgrade goreleaser to v2.8.1 Related to a regression inside v2.8.2 * chore: prepare release * docs: update changelog * docs: update GitHub Action assets (golangci#5724) * build(deps): bump github.com/ghostiam/protogetter from 0.3.13 to 0.3.14 (golangci#5727) * build(deps): bump mvdan.cc/gofumpt from 0.7.0 to 0.8.0 (golangci#5728) * build(deps): bump github.com/ldez/exptostd from 0.4.2 to 0.4.3 (golangci#5730) * build(deps): bump github.com/ldez/usetesting from 0.4.2 to 0.4.3 (golangci#5729) * dev: simplify mnd implementation (golangci#5731) * build(deps): bump github.com/ghostiam/protogetter from 0.3.14 to 0.3.15 (golangci#5732) * docs: fix default value of linters.exclusions.generated (golangci#5735) * chore: prepare release * docs: update GitHub Action assets (golangci#5738) * docs: update section about GoLand integration (golangci#5740) * fix: order of staticcheck settings during migration (golangci#5741) Co-authored-by: Fernandez Ludovic <[email protected]> * fix: add go.mod hash to the cache salt (golangci#5739) * dev: disable check-generated job (golangci#5742) * fix: related information position (golangci#5746) * docs: GoLand IDE only support v1 for now (golangci#5750) * fix: convert uint as pointer of uint for the migration (golangci#5755) * build(deps): bump go.augendre.info/fatcontext from 0.7.2 to 0.8.0 (golangci#5757) * docs: GoLand IDE support v2 (golangci#5758) * chore: prepare release * docs: update GitHub Action assets (golangci#5762) * chore: prepare release (golangci#5763) * docs: update GitHub Action assets (golangci#5764) * chore: prepare release (golangci#5765) * docs: update GitHub Action assets (golangci#5766) * docs: add note about golangci-lint v2 integration in VS Code (golangci#5768) * build(deps): bump go-simpler.org/musttag from 0.13.0 to 0.13.1 (golangci#5769) * build(deps): bump github.com/alecthomas/chroma/v2 from 2.16.0 to 2.17.0 (golangci#5772) * build(deps): bump github.com/tetafro/godot from 1.5.0 to 1.5.1 (golangci#5770) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump base-x from 3.0.9 to 3.0.11 in /docs (golangci#5776) * build(deps): bump the linter-testdata group across 2 directories with 2 updates (golangci#5777) * build(deps): bump github.com/shirou/gopsutil/v4 from 4.25.3 to 4.25.4 (golangci#5778) * build(deps): bump github.com/alecthomas/chroma/v2 from 2.17.0 to 2.17.2 (golangci#5779) * chore: prepare release * docs: update GitHub Action assets * build(deps): bump github.com/manuelarte/funcorder from 0.2.1 to 0.3.0 (golangci#5743) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump github.com/sonatard/noctx from 0.1.0 to 0.3.3 (golangci#5771) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump golangci/golangci-lint-action from 7.0.0 to 8.0.0 in the github-actions group (golangci#5780) * build(deps): bump golang.org/x/oauth2 from 0.29.0 to 0.30.0 in /scripts/gen_github_action_config in the scripts group (golangci#5781) * build(deps): bump github.com/jjti/go-spancheck from 0.6.4 to 0.6.5 (golangci#5784) * build(deps): bump golang.org/x/sys from 0.32.0 to 0.33.0 (golangci#5785) * build(deps): bump golang.org/x/tools from 0.32.0 to 0.33.0 (golangci#5786) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump github.com/ashanbrown/makezero from 1.2.0 to 2.0.1 (golangci#5782) * build(deps): bump github.com/ashanbrown/forbidigo from 1.6.0 to 2.1.0 (golangci#5783) * build(deps): bump github.com/securego/gosec/v2 from 2.22.3 to 2.22.4 (golangci#5788) * build(deps): bump github.com/manuelarte/funcorder from 0.3.0 to 0.5.0 (golangci#5792) * Add swaggo/swag formatter (golangci#5749) Co-authored-by: Fernandez Ludovic <[email protected]> * feat: add arangolint linter (golangci#5718) Co-authored-by: Fernandez Ludovic <[email protected]> * feat: add embeddedstructfieldcheck linter (golangci#5761) Co-authored-by: Fernandez Ludovic <[email protected]> * fix: exclusions path-expect (golangci#5798) * dev: simplify linter constructors (golangci#5796) * dev: remove AUR publishing (golangci#5799) * dev: restore aur-bin (golangci#5803) * chore: remove dead code (golangci#5801) * errcheck: add verbose option (golangci#5802) * docs: fix revive defer rule configuration (golangci#5804) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump github.com/Abirdcfly/dupword from 0.1.3 to 0.1.4 (golangci#5809) * build(deps): bump github.com/uudashr/iface from 1.3.1 to 1.3.2 (golangci#5810) * build(deps): bump github.com/alecthomas/chroma/v2 from 2.17.2 to 2.18.0 (golangci#5812) * build(deps): bump github.com/manuelarte/embeddedstructfieldcheck from 0.2.1 to 0.3.0 (golangci#5811) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump github.com/golangci/misspell from 0.6.0 to 0.7.0 (golangci#5813) * build(deps): bump the linter-testdata group across 2 directories with 2 updates (golangci#5814) Co-authored-by: Fernandez Ludovic <[email protected]> * dev: consistent version (golangci#5817) * docs: add description for linters.default sets (golangci#5818) * build(deps): bump github.com/uudashr/iface from 1.3.2 to 1.4.0 (golangci#5820) Co-authored-by: Fernandez Ludovic <[email protected]> * docs: improve typecheck FAQ (golangci#5821) Co-authored-by: Simon Sawert <[email protected]> * docs: note about plugin system (golangci#5822) * build(deps): bump github.com/jgautheron/goconst from 1.8.1 to 1.8.2 (golangci#5825) * fix: write the input to stdout when using stdin and there are no changes (golangci#5827) * build(deps): bump github.com/santhosh-tekuri/jsonschema/v6 from 6.0.1 to 6.0.2 (golangci#5829) * build(deps): bump github.com/sashamelentyev/usestdlibvars from 1.28.0 to 1.29.0 (golangci#5828) Co-authored-by: Fernandez Ludovic <[email protected]> * fix: error message when trying to migrate a migrated config (golangci#5836) Co-authored-by: Fernandez Ludovic <[email protected]> * fix: formatters CLI flags help message (golangci#5835) * build(deps): bump github.com/golangci/plugin-module-register from 0.1.1 to 0.1.2 (golangci#5838) * build(deps): bump github.com/mgechev/revive from 1.9.0 to 1.10.0 (golangci#5837) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump github.com/Abirdcfly/dupword from 0.1.4 to 0.1.5 (golangci#5839) * chore: improve pull request template * build(deps): bump github.com/Abirdcfly/dupword from 0.1.5 to 0.1.6 (golangci#5841) * Add noinlineerr linter (golangci#5826) * dev: some simplifications (golangci#5843) * dev: improve comments (golangci#5846) * build(deps): bump github.com/shirou/gopsutil/v4 from 4.25.4 to 4.25.5 (golangci#5849) * build(deps): bump the linter-testdata group across 4 directories with 4 updates (golangci#5850) * docs: improve module plugin explanation (golangci#5851) * dev: display list of staticcheck checks in debug (golangci#5853) * dev: correct debug log for default rules in revive (golangci#5852) * dev: use slice of pointers (golangci#5854) * dev: handle linter major versions (golangci#5848) * feat: deprecate print-resources-usage flag (golangci#5860) * docs: add a note about `GL_DEBUG=staticcheck` (golangci#5861) * fix: deduplicate typecheck errors (golangci#5864) * build(deps): bump golang.org/x/mod from 0.24.0 to 0.25.0 (golangci#5868) * build(deps): bump golang.org/x/tools from 0.33.0 to 0.34.0 (golangci#5867) * build(deps): bump github.com/ldez/gomoddirectives from 0.6.1 to 0.7.0 (golangci#5869) * dev: log level colors (golangci#5866) * docs: update gomoddirectives configuration * build(deps): bump github.com/ldez/exptostd from 0.4.3 to 0.4.4 (golangci#5876) * build(deps): bump github.com/ldez/usetesting from 0.4.3 to 0.5.0 (golangci#5877) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump brace-expansion from 1.1.11 to 1.1.12 in /docs (golangci#5878) * build(deps): bump github.com/securego/gosec/v2 from 2.22.4 to 2.22.5 (golangci#5880) * fix: typecheck memory leak (golangci#5884) * dev: remove useless condition * docs: update migration guide with --print-issued-lines (golangci#5886) * fix: stop the analysis after the first package analysis error (golangci#5885) * build(deps): bump github.com/go-viper/mapstructure/v2 from 2.2.1 to 2.3.0 (golangci#5888) * build(deps): bump brace-expansion from 1.1.11 to 1.1.12 in /.github/peril (golangci#5889) * fix: formats consistently the code with gci (golangci#5893) * build(deps): bump github.com/sonatard/noctx from 0.3.3 to 0.3.4 (golangci#5895) Co-authored-by: Fernandez Ludovic <[email protected]> * fix: unique version per custom build (golangci#5896) * build(deps): bump github.com/bombsimon/wsl/v5 from 4.7.0 to 5.0.0 (golangci#5900) Co-authored-by: Fernandez Ludovic <[email protected]> * dev: add suggested migration for linter configuration (golangci#5902) * chore: prepare release * docs: update documentation assets (golangci#5903) Co-authored-by: Fernandez Ludovic <[email protected]> * varnamelen: fix configuration (golangci#5904) * chore: prepare release * docs: update GitHub Action assets * dev: fix GitHub Action assets generation (golangci#5906) * build(deps): bump the linter-testdata group across 2 directories with 3 updates (golangci#5908) * docs: add the command to check the Go version used to build (golangci#5913) * build(deps): bump github.com/alecthomas/chroma/v2 from 2.18.0 to 2.19.0 (golangci#5914) * build(deps): bump github.com/shirou/gopsutil/v4 from 4.25.5 to 4.25.6 (golangci#5918) * godot: add noinline value into the JSONSchema (golangci#5922) * docs: fix tutorial URL (golangci#5925) * build(deps): bump golang.org/x/mod from 0.25.0 to 0.26.0 (golangci#5927) * build(deps): bump github.com/AlwxSin/noinlineerr from 1.0.3 to 1.0.4 (golangci#5928) * build(deps): bump golang.org/x/sys from 0.33.0 to 0.34.0 (golangci#5931) * docs: improve debug keys documentation (golangci#5930) * fix: panic: close of closed channel (golangci#5929) * chore: prepare release * docs: update GitHub Action assets * build(deps): bump github.com/uudashr/iface from 1.4.0 to 1.4.1 (golangci#5915) * build(deps): bump github.com/sonatard/noctx from 0.3.4 to 0.3.5 (golangci#5916) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump github.com/bombsimon/wsl/v5 from 5.0.0 to 5.1.0 (golangci#5917) * build(deps): bump github.com/nunnatsa/ginkgolinter from 0.19.1 to 0.20.0 (golangci#5932) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump golang.org/x/sync from 0.15.0 to 0.16.0 (golangci#5934) * build(deps): bump golang.org/x/tools from 0.34.0 to 0.35.0 (golangci#5935) * build(deps): bump github.com/mgechev/revive from 1.10.0 to 1.11.0 (golangci#5933) Co-authored-by: Fernandez Ludovic <[email protected]> * build(deps): bump go-simpler.org/sloglint from 0.11.0 to 0.11.1 (golangci#5936) * fix: panic: close of closed channel (golangci#5939) * build(deps): bump on-headers and compression in /docs (golangci#5944) * build(deps): bump github.com/go-viper/mapstructure/v2 from 2.3.0 to 2.4.0 (golangci#5947) * build(deps): bump github.com/spf13/pflag from 1.0.6 to 1.0.7 (golangci#5948) * build(deps): bump github.com/AlwxSin/noinlineerr from 1.0.4 to 1.0.5 (golangci#5949) * build(deps): bump github.com/securego/gosec/v2 from 2.22.5 to 2.22.6 (golangci#5950) * chore: prepare release * docs: update documentation assets (golangci#5951) Co-authored-by: Fernandez Ludovic <[email protected]> * docs: update GitHub Action assets (golangci#5952) * build(deps): bump github.com/securego/gosec/v2 from 2.22.6 to 2.22.7 (golangci#5953) * build(deps): bump form-data from 3.0.1 to 3.0.4 in /docs (golangci#5954) * tagliatelle: force upper case for custom initialisms (golangci#5956) * build(deps): bump github.com/daixiang0/gci from 0.13.6 to 0.13.7 (golangci#5957) * build(deps): bump github.com/ldez/grignotin from 0.9.0 to 0.10.0 (golangci#5958) * build(deps): bump github.com/bombsimon/wsl/v5 from 5.1.0 to 5.1.1 (golangci#5959) * build(deps): bump github.com/sonatard/noctx from 0.3.5 to 0.4.0 (golangci#5960) * build(deps): bump the linter-testdata group across 3 directories with 3 updates (golangci#5964) * chore: prepare release * chore(deps): update dependency go to v1.24.5 --------- Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Ludovic Fernandez <[email protected]> Co-authored-by: GolangCI-Lint Releaser <[email protected]> Co-authored-by: Oleksandr Redko <[email protected]> Co-authored-by: Matthew Gabeler-Lee <[email protected]> Co-authored-by: kasaikou <[email protected]> Co-authored-by: Gareth Jones <[email protected]> Co-authored-by: Tatsuya Kyushima <[email protected]> Co-authored-by: logica0419 <[email protected]> Co-authored-by: Manuel Doncel Martos <[email protected]> Co-authored-by: Chris Reeves <[email protected]> Co-authored-by: Nikita Mironov <[email protected]> Co-authored-by: sivchari <[email protected]> Co-authored-by: Tom Vendolsky <[email protected]> Co-authored-by: Cooper Benson <[email protected]> Co-authored-by: Gabriel Augendre <[email protected]> Co-authored-by: Manuel Doncel Martos <[email protected]> Co-authored-by: Takeo Kasai <[email protected]> Co-authored-by: Simon Sawert <[email protected]> Co-authored-by: ccoVeille <[email protected]> Co-authored-by: Alwx <[email protected]> Co-authored-by: Ville Skyttä <[email protected]> Co-authored-by: alaudaa-renovate[bot] <219066560+alaudaa-renovate[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The check was done in the wrong place, so it was not triggered
when the config was already migrated.
Another error message was reported about jsonschema validation issues.
Because a v2 config file does not validate against the v1 schema.
That was misleading.
Before
After