12b199c5e5
Noteable additions: - `redefines-builtin-id` forbid variable names that shadow go builtins - `empty-lines` remove unnecessary empty lines that `gofumpt` does not remove for some reason - `superfluous-else` eliminate more superfluous `else` branches Rules are also sorted alphabetically and I cleaned up various parts of `.golangci.yml`. (cherry picked from commit 74f0c84fa4245a20ce6fb87dac1faf2aeeded2a2) Conflicts: .golangci.yml apply the linter recommendations to Forgejo code as well |
||
---|---|---|
.. | ||
testdata/gitlab | ||
codebase.go | ||
codebase_test.go | ||
common.go | ||
dump.go | ||
error.go | ||
forgejo_downloader.go | ||
forgejo_downloader_test.go | ||
git.go | ||
gitbucket.go | ||
gitea_downloader.go | ||
gitea_downloader_test.go | ||
gitea_uploader.go | ||
gitea_uploader_test.go | ||
github.go | ||
github_test.go | ||
gitlab.go | ||
gitlab_test.go | ||
gogs.go | ||
gogs_test.go | ||
http_client.go | ||
main_test.go | ||
migrate.go | ||
migrate_test.go | ||
onedev.go | ||
onedev_test.go | ||
restore.go | ||
update.go |