1
0
Fork 0
forgejo/services/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
..
archiver
commitstatus fix: don't update cache for empty state 2024-04-16 12:02:01 +02:00
files Enable more revive linter rules (#30608) 2024-04-28 15:39:00 +02:00
adopt.go Enable more revive linter rules (#30608) 2024-04-28 15:39:00 +02:00
adopt_test.go
avatar.go
avatar_test.go
branch.go
cache.go
check.go
collaboration.go
collaboration_test.go
commit.go
contributors_graph.go Enable more revive linter rules (#30608) 2024-04-28 15:39:00 +02:00
contributors_graph_test.go
create.go
create_test.go
delete.go
delete_test.go
fork.go
fork_test.go
generate.go
generate_test.go
hooks.go
init.go
lfs.go
lfs_test.go
main_test.go
migrate.go
push.go
repository.go
repository_test.go
review.go
review_test.go
setting.go
template.go
transfer.go
transfer_test.go