Merge branch 'ci/run_readme_checks_not_in_parallel' into 'release/v5.1'

ci: run readme check when constants.py or check script updated (v5.1)

See merge request espressif/esp-idf!23386
This commit is contained in:
Fu Hanxi 2023-04-25 09:00:59 +08:00
commit 5bea8592d9
2 changed files with 6 additions and 2 deletions

View File

@ -139,7 +139,8 @@ repos:
name: Check if all apps readme files match given .build-test-rules.yml files. Modify the supported target tables
entry: tools/ci/check_build_test_rules.py check-readmes
language: python
files: 'tools/test_apps/.+|examples/.+|components/.+'
files: 'tools/test_apps/.+|examples/.+|components/.+|tools/idf_py_actions/constants.py|tools/ci/check_build_test_rules.py'
require_serial: true
additional_dependencies:
- PyYAML == 5.3.1
- idf_build_apps

View File

@ -1,6 +1,6 @@
#!/usr/bin/env python
# SPDX-FileCopyrightText: 2022 Espressif Systems (Shanghai) CO LTD
# SPDX-FileCopyrightText: 2022-2023 Espressif Systems (Shanghai) CO LTD
# SPDX-License-Identifier: Apache-2.0
import argparse
@ -468,6 +468,9 @@ if __name__ == '__main__':
else:
check_dirs.add(p)
if 'tools/idf_py_actions/constants.py' in arg.paths or 'tools/ci/check_build_test_rules.py' in arg.paths:
check_all = True
if check_all:
check_dirs = {IDF_PATH}
_exclude_dirs = [os.path.join(IDF_PATH, 'tools', 'unit-test-app'),