diff options
| author | Tristan Honscheid <honscheid@google.com> | 2023-05-16 14:33:26 -0600 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-05-16 22:40:35 +0000 |
| commit | b34dc2ae9022e2fbb57ae6477891ff32954c62fd (patch) | |
| tree | 31ffc2b523f6887313dd6bb823e02232cf9c39c2 /docs/chromeos-ec-firmware-test-requirements.md | |
| parent | 318306b3a82aa941abe0a559e87d66009f0663c9 (diff) | |
| download | chrome-ec-b34dc2ae9022e2fbb57ae6477891ff32954c62fd.tar.gz | |
presubmit: Require bug in LOW_COVERAGE_REASON
Enforce having a bug listed in the LOW_COVERAGE_REASON to track the
issue that impedes coverage for that specific CL. This encourages
developers to go back and fix those problems. Also update the
documentation.
BUG=None
TEST=util/check_low_coverage_reason.py
Change-Id: If35f918b11771d10944e0b6e7a4aeeba84e05eaf
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4537010
Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
Commit-Queue: Jeremy Bettis <jbettis@chromium.org>
Auto-Submit: Tristan Honscheid <honscheid@google.com>
Tested-by: Tristan Honscheid <honscheid@google.com>
Diffstat (limited to 'docs/chromeos-ec-firmware-test-requirements.md')
| -rw-r--r-- | docs/chromeos-ec-firmware-test-requirements.md | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/docs/chromeos-ec-firmware-test-requirements.md b/docs/chromeos-ec-firmware-test-requirements.md index 2e8b461eb6..721021275a 100644 --- a/docs/chromeos-ec-firmware-test-requirements.md +++ b/docs/chromeos-ec-firmware-test-requirements.md @@ -56,7 +56,8 @@ reports locally on your machine. In limited cases, you may amend your commit message to include the `LOW_COVERAGE_REASON` tag. This tag bypasses the code coverage requirement enforced by Gerrit. Simply add the tag followed by a description to justify -bypassing code coverage. +bypassing code coverage. You must include a reference to a bug (in the form +`b:1234567` or `b/1234567`) that tracks whatever issue is impeding coverage. ``` LOW_COVERAGE_REASON=no emulator for the ANX7483 exists b/248086547 |