This is an automated email from the ASF dual-hosted git repository.

cederom pushed a commit to branch releases/12.7
in repository https://gitbox.apache.org/repos/asf/nuttx.git

commit cc711e0c999dd89c0221b408e832ddbec7fb5ea9
Author: xuxingliang <xuxingli...@xiaomi.com>
AuthorDate: Fri Aug 9 15:46:16 2024 +0800

    tools: exit with error if py needs to format
    
    Make CI fail if format not pass.
    
    Signed-off-by: xuxingliang <xuxingli...@xiaomi.com>
---
 tools/checkpatch.sh | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/tools/checkpatch.sh b/tools/checkpatch.sh
index db3c48d5a9..992a87c3e9 100755
--- a/tools/checkpatch.sh
+++ b/tools/checkpatch.sh
@@ -83,9 +83,9 @@ check_file() {
   fi
 
   if [ ${@##*.} == 'py' ]; then
-    black --check $@
-    flake8 --config ${TOOLDIR}/../.github/linters/setup.cfg $@
-    isort $@
+    black --check $@ || fail=1
+    flake8 --config ${TOOLDIR}/../.github/linters/setup.cfg $@ || fail=1
+    isort $@ || fail=1
   elif [ "$(is_rust_file $@)" == "1" ]; then
     if ! command -v rustfmt &> /dev/null; then
       fail=1

Reply via email to