vocdoni / saas-backend / 29572380538

62%

main: 62%

Build Type

Pull #584

github

Committed by elboletaire

Commit Message

chore(lint): fix all golangci-lint issues for v2.12

Code fixes:

- convert stray fmt prints in db/ and migrations/ to structured log calls
- extract repeated validator description strings into shared constants
- replace sort.Strings/sort.Slice with slices.Sort/slices.SortFunc
- rename epoch var to startSec, add missing switch default case
- default oauthServiceURL to https (host serves TLS, http just redirects)
- nolint SA1019 in saltedkey (raw key mutation is the point of salted-key
  derivation) and parser.ParseDir in the errors AST test

Config changes:

- revive unhandled-error: allowlist fmt.Print*/Fprint* and in-memory
  writers (strings.Builder, bytes.Buffer) that never fail
- revive package-naming: exclude the intentional errors package
- goconst (1.10 extended detection): ignore test files and single-token
  strings, which are overwhelmingly bson field names and mongo operators

Pull Request Pull Request #584: chore(lint): fix all golangci-lint issues for v2.12

Coverage Stats

18 of 29 new or added lines in 10 files covered. (62.07%)

3 existing lines in 1 file now uncovered.

11491 of 18488 relevant lines covered (62.15%)

44.91 hits per line