From 5d50797dad179036cb0c0c3142b36a65b208ae22 Mon Sep 17 00:00:00 2001 From: Anas Nashif Date: Thu, 3 Dec 2020 15:54:40 -0500 Subject: [PATCH] ci: handle checpatch warnings as errors Fail CI if we have both errors and warnings. Signed-off-by: Anas Nashif --- scripts/ci/check_compliance.py | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/scripts/ci/check_compliance.py b/scripts/ci/check_compliance.py index 8c13539020e..02ddd07c6b0 100755 --- a/scripts/ci/check_compliance.py +++ b/scripts/ci/check_compliance.py @@ -213,11 +213,7 @@ class CheckPatch(ComplianceTest): except subprocess.CalledProcessError as ex: output = ex.output.decode("utf-8") - if re.search("[1-9][0-9]* errors,", output): - self.add_failure(output) - else: - # No errors found, but warnings. Show them. - self.add_info(output) + self.add_failure(output) class KconfigCheck(ComplianceTest):