From 91995914e5c2476bb8c5ad0b7a34e173b6da42d9 Mon Sep 17 00:00:00 2001 From: Xiangyu Wang Date: Wed, 29 Dec 2021 11:16:46 +0800 Subject: [PATCH] Add a code review rule (#14434) Signed-off-by: Xiangyu Wang --- CODE_REVIEW.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CODE_REVIEW.md b/CODE_REVIEW.md index f085b4909b..69648008c3 100644 --- a/CODE_REVIEW.md +++ b/CODE_REVIEW.md @@ -47,6 +47,7 @@ all Approvers are listed under OWNERS_ALIASES file. - If `[skip e2e]` is tagged, is it safe enough to skip running e2e tests? - Notes: it skips running e2e tests, if there is a `[skip e2e]` in the commit message. +- Does the code will generate similar logs many times in one seconds? ## Things to keep in mind when you are writing a review comment