github: workflows: Avoid listing tests manually
Instead of listing each test manually, use the script itself to list them: - When invoking it just exclude the one we don't want by default - When parsing the output files, use the -l output as a list Signed-off-by: Carles Cufi <carles.cufi@nordicsemi.no>
This commit is contained in:
parent
49b01b39b6
commit
5ed037023b
1 changed files with 7 additions and 6 deletions
13
.github/workflows/compliance.yml
vendored
13
.github/workflows/compliance.yml
vendored
|
@ -69,8 +69,7 @@ jobs:
|
|||
# debug
|
||||
ls -la
|
||||
git log --pretty=oneline | head -n 10
|
||||
./scripts/ci/check_compliance.py --annotate -m DevicetreeBindings
|
||||
-m Gitlint -m Identity -m Nits -m pylint -m checkpatch -m Kconfig \
|
||||
./scripts/ci/check_compliance.py --annotate -e KconfigBasic \
|
||||
-c origin/${BASE_REF}..
|
||||
|
||||
- name: upload-results
|
||||
|
@ -86,13 +85,15 @@ jobs:
|
|||
exit 1;
|
||||
fi
|
||||
|
||||
for file in Nits.txt checkpatch.txt Identity.txt Gitlint.txt pylint.txt Devicetree.txt Kconfig.txt DevicetreeBindings.txt; do
|
||||
if [[ -s $file ]]; then
|
||||
errors=$(cat $file)
|
||||
files=($(./scripts/ci/check_compliance.py -l))
|
||||
for file in "${files[@]}"; do
|
||||
f="${file}.txt"
|
||||
if [[ -s $f ]]; then
|
||||
errors=$(cat $f)
|
||||
errors="${errors//'%'/'%25'}"
|
||||
errors="${errors//$'\n'/'%0A'}"
|
||||
errors="${errors//$'\r'/'%0D'}"
|
||||
echo "::error file=${file}::$errors"
|
||||
echo "::error file=${f}::$errors"
|
||||
exit=1
|
||||
fi
|
||||
done
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue