Skip to content

Commit ce50334

Browse files
utzignvlsianpu
authored andcommitted
ci: relax signed-off-by checks on forks
Addresses issues when running signed-off-by checks on the Zephyr fork due to GH rebases done on UX rewriting the commit with the primary email instead of whatever email was used originally. Signed-off-by: Fabio Utzig <[email protected]>
1 parent 713bb79 commit ce50334

File tree

1 file changed

+10
-1
lines changed

1 file changed

+10
-1
lines changed

ci/check-signed-off-by.sh

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,16 @@ for sha in $commits; do
3232
lines="$(git show -s --format=%B ${sha})"
3333

3434
found_author=false
35-
found_committer=false
35+
# Don't enforce committer email on forks; this primarily avoids issues
36+
# running workflows on the zephyr fork, because rebases done in the GH UX
37+
# use the primary email of the committer, which might not match the one
38+
# used in git CLI.
39+
if [[ $GITHUB_REPOSITORY == mcu-tools/* ]]; then
40+
found_committer=false
41+
else
42+
found_committer=true
43+
fi
44+
3645
IFS=$'\n'
3746
for line in ${lines}; do
3847
stripped=$(echo $line | sed -e 's/^\s*//' | sed -e 's/\s*$//')

0 commit comments

Comments
 (0)