From d46c95b758b9fd67a38474d5e8cfbce6d2d12b70 Mon Sep 17 00:00:00 2001 From: Kevin O'Connor Date: Thu, 20 Oct 2022 11:56:55 -0400 Subject: [PATCH] docs: Add a note about fixing underlying bugs in CONTRIBUTING.md Signed-off-by: Kevin O'Connor --- docs/CONTRIBUTING.md | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/docs/CONTRIBUTING.md b/docs/CONTRIBUTING.md index 0c8b2c91..793fc7d5 100644 --- a/docs/CONTRIBUTING.md +++ b/docs/CONTRIBUTING.md @@ -62,6 +62,10 @@ Common things a reviewer will look for: Submissions must pass all [regression test cases](Debugging.md). + When fixing a defect in the code, submitters should have a general + understanding of the root cause of that defect, and the fix should + target that root cause. + Code submissions should not contain excessive debugging code, debugging options, nor run-time debug logging.