From 7cb8a821347bb630fdd2c89d4b043ffb466780a5 Mon Sep 17 00:00:00 2001 From: Ted Pudlik Date: Fri, 27 Dec 2024 12:04:27 -0800 Subject: [PATCH] docs: Update ref to No-Docs-Update-Reason MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Change-Id: I2e4ab5fc5dfc73316e1a2b8ae131a5427d02ff4a Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/256694 Pigweed-Auto-Submit: Ted Pudlik Lint: Lint 🤖 Commit-Queue: Auto-Submit Reviewed-by: Kayce Basques --- docs/contributing/index.rst | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/docs/contributing/index.rst b/docs/contributing/index.rst index dc6850ff1..91e7b3f7f 100644 --- a/docs/contributing/index.rst +++ b/docs/contributing/index.rst @@ -189,12 +189,8 @@ Submission checklist All Pigweed changes must either: #. Include updates to documentation, or -#. Include ``No-Docs-Update-Reason: `` in a Gerrit comment on the CL. - For example: - - * ``No-Docs-Update-Reason: formatting tweaks`` - * ``No-Docs-Update-Reason: internal cleanups`` - * ``No-Docs-Update-Reason: bugfix`` +#. Assert no documentation is needed by voting +1 on the ``Docs-Not-Needed`` + #label in Gerrit. It's acceptable to only document new changes in an otherwise underdocumented module, but it's not acceptable to not document new changes because the module