From 448b74840e26f7598d7b22420089c9c26a5d38ef Mon Sep 17 00:00:00 2001 From: Matthias Date: Sat, 20 Apr 2024 09:09:42 +0200 Subject: [PATCH] Don't use mutable default arguments to functions --- freqtrade/resolvers/iresolver.py | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/freqtrade/resolvers/iresolver.py b/freqtrade/resolvers/iresolver.py index 6295e845d..bcbb5704b 100644 --- a/freqtrade/resolvers/iresolver.py +++ b/freqtrade/resolvers/iresolver.py @@ -47,7 +47,7 @@ class IResolver: @classmethod def build_search_paths(cls, config: Config, user_subdir: Optional[str] = None, - extra_dirs: List[str] = []) -> List[Path]: + extra_dirs: Optional[List[str]] = None) -> List[Path]: abs_paths: List[Path] = [] if cls.initial_search_path: @@ -57,8 +57,9 @@ class IResolver: abs_paths.insert(0, config['user_data_dir'].joinpath(user_subdir)) # Add extra directory to the top of the search paths - for dir in extra_dirs: - abs_paths.insert(0, Path(dir).resolve()) + if extra_dirs: + for dir in extra_dirs: + abs_paths.insert(0, Path(dir).resolve()) if cls.extra_path and (extra := config.get(cls.extra_path)): abs_paths.insert(0, Path(extra).resolve()) @@ -139,7 +140,7 @@ class IResolver: @classmethod def _load_object(cls, paths: List[Path], *, object_name: str, add_source: bool = False, - kwargs: dict = {}) -> Optional[Any]: + kwargs: Dict) -> Optional[Any]: """ Try to load object from path list. """