forgejo/modules/repository
silverwind 12b199c5e5
Enable more revive linter rules (#30608)
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
2024-04-28 15:39:00 +02:00
..
branch.go
collaborator.go
collaborator_test.go
commits.go
commits_test.go
create.go
create_test.go
delete.go
env.go
fork.go
hooks.go
init.go
init_test.go
license.go
license_test.go
main_test.go
push.go
repo.go
repo_test.go
temp.go Enable more revive linter rules (#30608) 2024-04-28 15:39:00 +02:00