ci(lint): drop revive exported rule; require no doc-comment boilerplate
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.
This commit is contained in:
+5
-2
@@ -16,10 +16,13 @@ linters:
|
|||||||
|
|
||||||
linters-settings:
|
linters-settings:
|
||||||
revive:
|
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:
|
rules:
|
||||||
- name: var-naming
|
- name: var-naming
|
||||||
- name: exported
|
- name: unused-parameter
|
||||||
arguments: ["checkPrivateReceivers", "sayRepetitiveInsteadOfStutters"]
|
- name: early-return
|
||||||
|
|
||||||
issues:
|
issues:
|
||||||
exclude-use-default: false
|
exclude-use-default: false
|
||||||
|
|||||||
Reference in New Issue
Block a user