From c88f20859aa20f32b17e7f728195f31d6146bbfb Mon Sep 17 00:00:00 2001 From: Joscha Date: Sun, 23 May 2021 11:04:50 +0200 Subject: [PATCH] Explain config file dumping --- PFERD/__main__.py | 2 ++ PFERD/config.py | 10 ++++++++-- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/PFERD/__main__.py b/PFERD/__main__.py index 5cc6ef6..2578487 100644 --- a/PFERD/__main__.py +++ b/PFERD/__main__.py @@ -81,6 +81,8 @@ def configure_logging_from_config(args: argparse.Namespace, config: Config) -> N def dump_config(args: argparse.Namespace, config: Config) -> None: + log.explain_topic("Dumping config") + try: if args.dump_config is True: config.dump() diff --git a/PFERD/config.py b/PFERD/config.py index 26a9eb6..e68db53 100644 --- a/PFERD/config.py +++ b/PFERD/config.py @@ -5,6 +5,8 @@ from configparser import ConfigParser, SectionProxy from pathlib import Path from typing import Any, List, NoReturn, Optional, Tuple +from rich.markup import escape + from .logging import log from .utils import prompt_yes_no @@ -122,10 +124,14 @@ class Config: May throw a ConfigDumpError. """ - if not path: + if path: + log.explain("Using custom path") + else: + log.explain("Using default path") path = self._default_path() - print(f"Dumping config to {path}") + log.explain(f"Dumping to {str(path.absolute())!r}") + log.print(f"[bold bright_cyan]Dumping[/] to {escape(repr(str(path.absolute())))}") try: path.parent.mkdir(parents=True, exist_ok=True)