michaelh/zephyr
1
0
Fork 0
Code Releases Activity

ci: compliance: fail on errors

Browse source 
Fail workflow in case of script crashes.

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
This commit is contained in:
Anas Nashif 2021-01-20 08:46:47 -05:00
commit f9b76ca76e
1 changed files with 6 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

7
.github/workflows/compliance.yml vendored
View file

@ -36,6 +36,7 @@ jobs:
west update west update
- name: Run Compliance Tests - name: Run Compliance Tests
continue-on-error: true
id: compliance id: compliance
env: env:
BASE_REF: ${{ github.base_ref }} BASE_REF: ${{ github.base_ref }}
@ -48,7 +49,7 @@ jobs:
# debug # debug
ls -la ls -la
git log --pretty=oneline | head -n 10 git log --pretty=oneline | head -n 10
./scripts/ci/check_compliance.py -m Codeowners -m Devicetree -m Gitlint -m Identity -m Nits -m pylint -m checkpatch -m Kconfig -c origin/${BASE_REF}.. || true ./scripts/ci/check_compliance.py -m Codeowners -m Devicetree -m Gitlint -m Identity -m Nits -m pylint -m checkpatch -m Kconfig -c origin/${BASE_REF}..
- name: upload-results - name: upload-results
uses: actions/upload-artifact@master uses: actions/upload-artifact@master
@ -59,6 +60,10 @@ jobs:
- name: check-warns - name: check-warns
run: | run: |
if [[ ! -s "compliance.xml" ]]; then
exit 1;
fi
for file in Nits.txt checkpatch.txt Identity.txt Gitlint.txt pylint.txt Devicetree.txt Kconfig.txt Codeowners.txt; do for file in Nits.txt checkpatch.txt Identity.txt Gitlint.txt pylint.txt Devicetree.txt Kconfig.txt Codeowners.txt; do
if [[ -s $file ]]; then if [[ -s $file ]]; then
errors=$(cat $file) errors=$(cat $file)