uranusjr commented on code in PR #22051:
URL: https://github.com/apache/airflow/pull/22051#discussion_r848830094
##########
airflow/utils/file.py:
##########
@@ -108,46 +194,77 @@ def open_maybe_zipped(fileloc, mode='r'):
return open(fileloc, mode=mode)
-def find_path_from_directory(base_dir_path: str, ignore_file_name: str) ->
Generator[str, None, None]:
+def _find_path_from_directory(
+ base_dir_path: str,
+ ignore_file_name: str,
+ ignore_rule_type: Type[_IgnoreRule],
+) -> Generator[str, None, None]:
"""
- Search the file and return the path of the file that should not be ignored.
- :param base_dir_path: the base path to be searched for.
- :param ignore_file_name: the file name in which specifies a regular
expression pattern is written.
+ Recursively search the base path and return the list of file paths that
should not be ignored by
+ regular expressions in any ignore files at each directory level.
+ :param base_dir_path: the base path to be searched
+ :param ignore_file_name: the file name containing regular expressions for
files that should be ignored.
+ :param ignore_rule_type: the concrete class for ignore rules, which
implements the _IgnoreRule interface.
- :return : file path not to be ignored.
+ :return : a generator of file paths which should not be ignored.
Review Comment:
```suggestion
:return: a generator of file paths which should not be ignored.
```
Shouldn’t have that space.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
