diff --git a/CMakeLists.txt b/CMakeLists.txt index ce8e677d7eb..b471f65473b 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -1,5 +1,6 @@ # SPDX-License-Identifier: Apache-2.0 + # *DOCUMENTATION* # # Note that this is *NOT* the top-level CMakeLists.txt. That's in the diff --git a/scripts/ci/check_compliance.py b/scripts/ci/check_compliance.py index 0b82bcce456..e22ec86bd33 100755 --- a/scripts/ci/check_compliance.py +++ b/scripts/ci/check_compliance.py @@ -1456,17 +1456,24 @@ class Identity(ComplianceTest): def run(self): for shaidx in get_shas(COMMIT_RANGE): - auth_name, auth_email, body = git( - 'show', '-s', '--format=%an%n%ae%n%b', shaidx - ).split('\n', 2) + commit_info = git('show', '-s', '--format=%an%n%ae%n%b', shaidx).split('\n', 2) + + failures = [] + + if len(commit_info) == 2: + failures.append(f'{shaidx}: Empty commit message body') + auth_name, auth_email = commit_info + body = '' + elif len(commit_info) == 3: + auth_name, auth_email, body = commit_info + else: + self.failure(f'Unable to parse commit message for {shaidx}') match_signoff = re.search(r"signed-off-by:\s(.*)", body, re.IGNORECASE) detailed_match = re.search(r"signed-off-by:\s(.*) <(.*)>", body, re.IGNORECASE) - failures = [] - if auth_email.endswith("@users.noreply.github.com"): failures.append(f"{shaidx}: author email ({auth_email}) must " "be a real email and cannot end in "