Skip to content

Commit 61decf1

Browse files
authored
Small change to parameters of load_config (#664)
Pass need parameters instead of the argparse args object. Signed-off-by: Eric Brown <eric.brown@securesauce.dev>
1 parent 5f1029e commit 61decf1

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

precli/cli/main.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -166,12 +166,12 @@ def setup_arg_parser():
166166
return args
167167

168168

169-
def load_config(args) -> dict:
170-
if args.config:
171-
return tomllib.load(args.config)
169+
def load_config(config: dict, targets: list[str]) -> dict:
170+
if config:
171+
return tomllib.load(config)
172172
else:
173173
default_confs = (".precli.toml", "precli.toml", "pyproject.toml")
174-
for target in filter(os.path.isdir, args.targets):
174+
for target in filter(os.path.isdir, targets):
175175
for conf in default_confs:
176176
path = pathlib.Path(target) / conf
177177
if path.exists():
@@ -367,7 +367,7 @@ def main():
367367
args = setup_arg_parser()
368368

369369
# Load optional configuration file
370-
config = load_config(args)
370+
config = load_config(args.config, args.targets)
371371

372372
# CLI enabled/disabled override any config in files
373373
config["enabled"] = (

0 commit comments

Comments
 (0)