michaelh/zephyr
1
0
Fork 0
Code Releases Activity

scripts: ci: check_compliance: fix identity check for multiple DCOs

Browse source 
The current implementation of the identity check fails if multiple DCO
signoff lines are present and the first instance of the signoff line
does not belong to the author of the commit. This patch proposes a solution
that allows patches with multiple DCO signoff lines to pass the identity
check, regardless of the order of the signoff lines.

Signed-off-by: Mirai SHINJO <oss@mshinjo.com>
This commit is contained in:
Mirai SHINJO 2025-05-03 21:37:38 -04:00 • committed by Benjamin Cabé
commit e4650bc349
1 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
scripts/ci/check_compliance.py
View file

@ -1462,7 +1462,8 @@ class Identity(ComplianceTest):
match_signoff = re.search(r"signed-off-by:\s(.*)", body,
re.IGNORECASE)
detailed_match = re.search(r"signed-off-by:\s(.*) <(.*)>", body,
detailed_match = re.search(rf"signed-off-by:\s({re.escape(auth_name)}) <({re.escape(auth_email)})>",
body,
re.IGNORECASE)
failures = []