Merge branch 'bugfix/ci_check_executable_windows_v4.3' into 'release/v4.3'

ci: fix check_executables issue on Windows (v4.3)

See merge request espressif/esp-idf!20094

.pre-commit-config.yaml

@@ -22,8 +22,6 @@ repos:
       - id: end-of-file-fixer
         exclude: *whitespace_excludes
       - id: check-executables-have-shebangs
-      - id: file-contents-sorter
-        files: 'tools/ci/executable-list.txt'
       - id: mixed-line-ending
         args: ['-f=lf']
       - id: double-quote-string-fixer
@@ -75,3 +73,8 @@ repos:
         language: python
         files: '\.gitlab/CODEOWNERS'
         pass_filenames: false
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v3.4.0
+    hooks:
+      - id: file-contents-sorter
+        files: 'tools/ci/executable-list.txt'

tools/ci/idf_ci_utils.py

@@ -55,8 +55,8 @@ def _check_git_filemode(full_path):  # type: (str) -> bool
 
     mode = stdout.split(' ', 1)[0]  # e.g. 100644 for a rw-r--r--
     if any([int(i, 8) & 1 for i in mode[-3:]]):
-        return False
-    return True
+        return True
+    return False
 
 
 def is_executable(full_path):  # type: (str) -> bool