Re: [bluez/action-ci] c1b86b: ci: add verify_fixes and verify_signedoff checks f...

[Bastien Nocera <hadess@hadess.net>]

On Thu, 2026-05-21 at 08:48 -0700, Luiz Augusto von Dentz wrote:
>   Branch: refs/heads/main
>   Home:   https://github.com/bluez/action-ci
>   Commit: c1b86bca7cacc936bea82b3ca8b2ee4b4fcb6e74
>      
> https://github.com/bluez/action-ci/commit/c1b86bca7cacc936bea82b3ca8b2ee4b4fcb6e74
>   Author: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
>   Date:   2026-05-21 (Thu, 21 May 2026)
> 
>   Changed paths:
>     M Dockerfile
>     M ci.py
>     M ci/__init__.py
>     A ci/verifyfixes.py
>     A ci/verifysignedoff.py
>     A scripts/verify_fixes.sh
>     A scripts/verify_signedoff.sh
> 
>   Log Message:
>   -----------
>   ci: add verify_fixes and verify_signedoff checks for kernel patches
> 
> Add scripts from gregkh/gregkh-linux adapted for GitHub Actions:
> - verify_fixes.sh: validates Fixes: tag format, SHA existence,
> subject
>   match, and ancestry (removed external Linus tree dependency)
> - verify_signedoff.sh: validates author/committer Signed-off-by
> presence

I haven't verified whether the functionality matches 1-1, but gitlint
already has support for verifying signed-off-by:

$ gitlint -C gitlint --contrib contrib-body-requires-signed-off-by --
msg-filename example.msg
1: CC1 Body does not contain a 'Signed-off-by' line

I've tried to integrate it into:
https://github.com/bluez/action-ci/pull/6

> 
> Both run against origin/master..HEAD after checkpatch in the kernel
> CI
> pipeline and report results as warnings to patchwork.
> 
> 
> 
> To unsubscribe from these emails, change your notification settings
> at https://github.com/bluez/action-ci/settings/notifications