silverwind 74f0c84fa4
Enable more `revive` linter rules ()
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`.
..
v102.go Implement FSFE REUSE for golang files ()
v103.go Rename `Sync2` -> `Sync` ()
v104.go Rename `Sync2` -> `Sync` ()
v105.go Rename `Sync2` -> `Sync` ()
v106.go Rename `Sync2` -> `Sync` ()
v107.go Rename `Sync2` -> `Sync` ()
v108.go Rename `Sync2` -> `Sync` ()
v109.go Rename `Sync2` -> `Sync` ()
v110.go Implement FSFE REUSE for golang files ()
v111.go Enable more `revive` linter rules ()
v112.go Fix all possible setting error related storages and added some tests ()
v113.go Rename `Sync2` -> `Sync` ()
v114.go Implement FSFE REUSE for golang files ()
v115.go Fix all possible setting error related storages and added some tests ()
v116.go Rename `Sync2` -> `Sync` ()