forked from espressif/esp-idf
Merge branch 'ci/fix_get_mr_component' into 'master'
ci: fix get mr components, stop printing out debugging info See merge request espressif/esp-idf!31162
This commit is contained in:
@@ -7,6 +7,7 @@
|
|||||||
# SPDX-License-Identifier: Apache-2.0
|
# SPDX-License-Identifier: Apache-2.0
|
||||||
#
|
#
|
||||||
import argparse
|
import argparse
|
||||||
|
import logging
|
||||||
import os
|
import os
|
||||||
import subprocess
|
import subprocess
|
||||||
import typing as t
|
import typing as t
|
||||||
@@ -81,10 +82,10 @@ def get_modified_component(filepath: str) -> t.Optional[str]:
|
|||||||
for part in f_path.parts[1:]:
|
for part in f_path.parts[1:]:
|
||||||
if component_parent_dirs[-1] == 'components' or component_parent_dirs[-1].endswith('common_components'):
|
if component_parent_dirs[-1] == 'components' or component_parent_dirs[-1].endswith('common_components'):
|
||||||
if part not in _COMPONENT_NAME_DIR_RECORDS:
|
if part not in _COMPONENT_NAME_DIR_RECORDS:
|
||||||
print('Found component "%s" in path "%s"' % (part, component_parent_dirs))
|
logging.debug('Found component "%s" in path "%s"' % (part, component_parent_dirs))
|
||||||
_COMPONENT_NAME_DIR_RECORDS[part] = component_parent_dirs
|
_COMPONENT_NAME_DIR_RECORDS[part] = component_parent_dirs
|
||||||
elif _COMPONENT_NAME_DIR_RECORDS.get(part) != component_parent_dirs:
|
elif _COMPONENT_NAME_DIR_RECORDS.get(part) != component_parent_dirs:
|
||||||
print(
|
logging.debug(
|
||||||
'WARNING!!! Found component "%s" in path "%s" and "%s"'
|
'WARNING!!! Found component "%s" in path "%s" and "%s"'
|
||||||
% (part, component_parent_dirs, _COMPONENT_NAME_DIR_RECORDS.get(part))
|
% (part, component_parent_dirs, _COMPONENT_NAME_DIR_RECORDS.get(part))
|
||||||
)
|
)
|
||||||
|
Reference in New Issue
Block a user