chore(revive): use a stricter revive config

(But also ignore optionparser for now.)

Makefile

@@ -18,7 +18,7 @@ staticcheck: fmt
 	staticcheck ./...
 
 revive: fmt
-	revive -config revive.toml ./...
+	revive -config revive.toml -exclude internal/optionparser ./...
 
 test:
 	go test -shuffle on github.com/telemachus/gitmirror/internal/cli

revive.toml

@@ -1,30 +1,23 @@
 ignoreGeneratedHeader = false
 severity = "warning"
 confidence = 0.8
-errorCode = 0
+errorCode = 1
 warningCode = 0
 
-[rule.blank-imports]
-[rule.context-as-argument]
-[rule.context-keys-type]
-[rule.dot-imports]
-[rule.error-return]
-[rule.error-strings]
-[rule.error-naming]
-[rule.exported]
-[rule.if-return]
-[rule.increment-decrement]
-[rule.var-naming]
-[rule.var-declaration]
-[rule.package-comments]
-[rule.range]
-[rule.receiver-naming]
-[rule.time-naming]
-[rule.unexported-return]
-[rule.indent-error-flow]
-[rule.errorf]
-[rule.empty-block]
-[rule.superfluous-else]
-[rule.unused-parameter]
-[rule.unreachable-code]
-[rule.redefines-builtin-id]
+enableAllRules = true
+
+[rule.add-constant]
+  Disabled = true
+
+[rule.deep-exit]
+  Disabled = true
+
+[rule.flag-parameter]
+  Disabled = true
+
+[rule.line-length-limit]
+  Arguments = [100]
+
+[rule.unhandled-error]
+  Arguments = ["fmt.Printf", "fmt.Fprintf", "fmt.Println", "fmt.Fprintln"]
+