From 743fbe9d5c151e4af3af7c010b3e31ff2f9b5f0c Mon Sep 17 00:00:00 2001 From: bvandeusen Date: Sat, 18 Apr 2026 21:47:40 +0000 Subject: [PATCH] ci(lint): drop revive `exported` rule; require no doc-comment boilerplate MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Revive's `exported` check forced doc comments on every exported symbol — that conflicts with the project's policy of only writing comments when the "why" is non-obvious. Keep `var-naming`, `unused-parameter`, and `early-return` for signal that doesn't mandate prose. --- .golangci.yml | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/.golangci.yml b/.golangci.yml index 7b87777e..f89576e3 100644 --- a/.golangci.yml +++ b/.golangci.yml @@ -16,10 +16,13 @@ linters: linters-settings: revive: + # Intentionally narrow: we skip `exported` (no doc-comment requirement) per + # the project's no-boilerplate-comment policy. Re-enable if the public API + # surface grows to the point where documentation lives alongside it. rules: - name: var-naming - - name: exported - arguments: ["checkPrivateReceivers", "sayRepetitiveInsteadOfStutters"] + - name: unused-parameter + - name: early-return issues: exclude-use-default: false