Claudio Bantaloukas (rdfm) <[email protected]> approved the changes:
LGTM, once the tests have been renamed to be consistent with the script files 
being called.

> +++ .forgejo/workflows/format-checks.yaml
> @@ -34,2 +30,3 @@
>            done
> -          echo "styleresult=pass" | tee -a $FORGEJO_OUTPUT
> +
> +  ChangeLog-format-check:
Please call it git_check_commit like the program being called.

> +++ .forgejo/workflows/format-checks.yaml
> @@ -4,3 +4,3 @@
>  
>  jobs:
> -  format-checks:
> +  GNU-style-check:
Please call it check_GNU_style like the program being called.


--
https://forge.sourceware.org/gcc/gcc-TEST/pulls/169#issuecomment-6195

Reply via email to