Skip to content

Commit

Permalink
CI: compliance: fail on errors
Browse files Browse the repository at this point in the history
Fail workflow in case of script crashes.

This patch is a cherry-pick of a zephyr patch implemented by
Anas Nashif:

zephyrproject-rtos/zephyr#31452

Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
  • Loading branch information
arnopo committed Apr 26, 2021
1 parent ad87802 commit 56ac238
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion .github/workflows/compliance.yml
Expand Up @@ -32,7 +32,7 @@ jobs:
git config --global user.name "Your Name"
git rebase origin/${BASE_REF}
export
./scripts/ci/check_compliance.py -m checkpatch -m Gitlint -m Identity -c origin/${BASE_REF}.. || true
./scripts/ci/check_compliance.py -m checkpatch -m Gitlint -m Identity -c origin/${BASE_REF}..
- name: upload-results
uses: actions/upload-artifact@master
Expand All @@ -43,6 +43,9 @@ jobs:

- name: check-warns
run: |
if [[ ! -s "compliance.xml" ]]; then
exit 1;
fi
if [ -s checkpatch.txt ]; then
errors=$(cat checkpatch.txt)
errors="${errors//'%'/'%25'}"
Expand Down

0 comments on commit 56ac238

Please sign in to comment.