Skip to content

Commit

Permalink
Merge branch 'bugfix/ci_check_executable_windows' into 'master'
Browse files Browse the repository at this point in the history
ci: fix check_executables issue on Windows

See merge request espressif/esp-idf!13342
  • Loading branch information
hfudev committed Apr 27, 2021
2 parents f975329 + 7f2973f commit cba0725
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 7 deletions.
7 changes: 5 additions & 2 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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|tools\/ci\/mypy_ignore_list\.txt)'
- id: mixed-line-ending
args: ['-f=lf']
- id: double-quote-string-fixer
Expand Down Expand Up @@ -97,3 +95,8 @@ repos:
additional_dependencies: ['mypy==0.800', 'mypy-extensions==0.4.3']
language: python
types: [python]
- 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\/mypy_ignore_list\.txt)'
8 changes: 4 additions & 4 deletions tools/ci/idf_ci_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -56,8 +56,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
Expand Down Expand Up @@ -90,7 +90,7 @@ def get_git_files(path=IDF_PATH, full_path=False): # type: (str, bool) -> list[
# https://github.com/python/cpython/pull/6299/commits/bfd63120c18bd055defb338c075550f975e3bec1
# In order to solve python https://bugs.python.org/issue9584
# glob pattern does not support brace expansion issue
def _translate(pat):
def _translate(pat): # type: (str) -> str
"""Translate a shell PATTERN to a regular expression.
There is no way to quote meta-characters.
"""
Expand Down Expand Up @@ -179,7 +179,7 @@ def _translate(pat):
return res


def translate(pat):
def translate(pat): # type: (str) -> str
res = _translate(pat)
return r'(?s:%s)\Z' % res

Expand Down
1 change: 0 additions & 1 deletion tools/ci/mypy_ignore_list.txt
Original file line number Diff line number Diff line change
Expand Up @@ -171,7 +171,6 @@ tools/ci/checkout_project_ref.py
tools/ci/ci_fetch_submodule.py
tools/ci/deploy_docs.py
tools/ci/envsubst.py
tools/ci/idf_ci_utils.py
tools/ci/normalize_clangtidy_path.py
tools/ci/python_packages/gitlab_api.py
tools/ci/python_packages/idf_http_server_test/adder.py
Expand Down

0 comments on commit cba0725

Please sign in to comment.