mirror of
1
Fork 0
forgejo/routers/api/packages/container
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
..
auth.go Remove `Named` interface (#26913) 2023-09-05 15:58:30 +00:00
blob.go Remove `db.DefaultContext` in `routers/` and `cmd/` (#26076) 2023-07-23 23:47:27 -04:00
container.go Enable more `revive` linter rules (#30608) 2024-04-28 15:39:00 +02:00
errors.go Implement FSFE REUSE for golang files (#21840) 2022-11-27 18:20:29 +00:00
manifest.go Move notification interface to services layer (#26915) 2023-09-05 18:37:47 +00:00