From 95053c83589a1ee7762c22ed91426fa87acc02db Mon Sep 17 00:00:00 2001 From: Carles Cufi Date: Mon, 21 Nov 2022 11:09:49 +0100 Subject: [PATCH] scripts: compliance: Fix DT Bindings check name Check names must not contain spaces, since they are passed via `-m` in the command-line. Also update .gitignore and compliance.yml. Signed-off-by: Carles Cufi --- .github/workflows/compliance.yml | 6 +++--- .gitignore | 1 + scripts/ci/check_compliance.py | 2 +- 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/.github/workflows/compliance.yml b/.github/workflows/compliance.yml index cacd7a9c4c2..3c6ff2ede26 100644 --- a/.github/workflows/compliance.yml +++ b/.github/workflows/compliance.yml @@ -69,8 +69,8 @@ jobs: # debug ls -la git log --pretty=oneline | head -n 10 - ./scripts/ci/check_compliance.py --annotate -m Devicetree -m Gitlint \ - -m Identity -m Nits -m pylint -m checkpatch -m Kconfig \ + ./scripts/ci/check_compliance.py --annotate -m DevicetreeBindings + -m Gitlint -m Identity -m Nits -m pylint -m checkpatch -m Kconfig \ -c origin/${BASE_REF}.. - name: upload-results @@ -86,7 +86,7 @@ jobs: exit 1; fi - for file in Nits.txt checkpatch.txt Identity.txt Gitlint.txt pylint.txt Devicetree.txt Kconfig.txt; do + 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) errors="${errors//'%'/'%25'}" diff --git a/.gitignore b/.gitignore index 085914ebbaa..b8121c11784 100644 --- a/.gitignore +++ b/.gitignore @@ -73,3 +73,4 @@ Codeowners.txt Nits.txt pylint.txt Identity.txt +DevicetreeBindings.txt diff --git a/scripts/ci/check_compliance.py b/scripts/ci/check_compliance.py index e90e393fbc5..4eba67926a7 100755 --- a/scripts/ci/check_compliance.py +++ b/scripts/ci/check_compliance.py @@ -224,7 +224,7 @@ class DevicetreeBindingsCheck(ComplianceTest): """ Checks if we are introducing any unwanted properties in Devicetree Bindings. """ - name = "Devicetree Bindings" + name = "DevicetreeBindings" doc = "See https://docs.zephyrproject.org/latest/build/dts/bindings.html for more details." path_hint = ZEPHYR_BASE