ci: compliance: in case of too many errors, send as a single failure
Do not try to format each single failure, this will take too long to process and will run for hours and spam GH. Signed-off-by: Anas Nashif <anas.nashif@intel.com>
This commit is contained in:
parent
ceaf6055e1
commit
420ffdd8a8
1 changed files with 7 additions and 0 deletions
|
@ -221,6 +221,13 @@ class CheckPatch(ComplianceTest):
|
||||||
r'^\s*#(\d+):\s*FILE:\s*(.+):(\d+):'
|
r'^\s*#(\d+):\s*FILE:\s*(.+):(\d+):'
|
||||||
|
|
||||||
matches = re.findall(regex, output, re.MULTILINE)
|
matches = re.findall(regex, output, re.MULTILINE)
|
||||||
|
|
||||||
|
# add a guard here for excessive number of errors, do not try and
|
||||||
|
# process each one of them and instead push this as one failure.
|
||||||
|
if len(matches) > 500:
|
||||||
|
self.failure(output)
|
||||||
|
return
|
||||||
|
|
||||||
for m in matches:
|
for m in matches:
|
||||||
self.fmtd_failure(m[1].lower(), m[2], m[5], m[6], col=None,
|
self.fmtd_failure(m[1].lower(), m[2], m[5], m[6], col=None,
|
||||||
desc=m[3])
|
desc=m[3])
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue