mirror of
1
Fork 0
forgejo/services/gitdiff
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
..
testdata/academic-module
csv.go Use Go 1.21 and update dependencies (#26878) 2023-09-03 10:34:57 +00:00
csv_test.go Even more `db.DefaultContext` refactor (#27352) 2023-10-03 10:30:41 +00:00
gitdiff.go Enable more `revive` linter rules (#30608) 2024-04-28 15:39:00 +02:00
gitdiff_test.go [BUG] split code conversations in diff tab (#2306) 2024-02-16 12:16:11 +00:00
highlightdiff.go Add option to disable ambiguous unicode characters detection (#28454) 2023-12-17 14:38:54 +00:00
highlightdiff_test.go Implement FSFE REUSE for golang files (#21840) 2022-11-27 18:20:29 +00:00
main_test.go make writing main test easier (#27270) 2023-09-28 01:38:53 +00:00