diff options
| author | ache <ache@ache.one> | 2026-01-23 13:37:10 +0100 |
|---|---|---|
| committer | ache <ache@ache.one> | 2026-01-23 13:37:10 +0100 |
| commit | ae6f8c5550ca8e4bf58ecf0bbfecc8a9dcbcd43d (patch) | |
| tree | 59051e645905ef008a9c86903f3e9a8f78953ef3 | |
| parent | fix: Improves error messages for paste retrieval failures (diff) | |
fix: Prints error messages to stderr
This commit changes several print statements in the CLI to output to `sys.stderr` instead of `sys.stdout`. This ensures that error and warning messages are not mixed with normal output, improving clarity and making it easier to pipe the output of the CLI to other tools. The affected areas include config file warnings, multiple action errors, and message displays.
| -rw-r--r-- | src/lolbin/cli.py | 19 |
1 files changed, 11 insertions, 8 deletions
diff --git a/src/lolbin/cli.py b/src/lolbin/cli.py index c53bd27..f976e54 100644 --- a/src/lolbin/cli.py +++ b/src/lolbin/cli.py @@ -82,7 +82,10 @@ def get_config(): if not config_path.exists(): print("Config file doesn't exists.", file=sys.stderr) # WARN: That is false, should return empty config and a link to the documentation section "config.toml". - print("Configure lolbin <https://source.tube/ache/lolbin#configuration>") + print( + "Configure lolbin <https://source.tube/ache/lolbin#configuration>", + file=sys.stderr, + ) return {} with open(config_path, "rb") as f: @@ -236,7 +239,7 @@ def show_paste(paste, address): if "response" in data: if "message" in data["response"]: - print(f"Message: {data['response']['message']}") + print(f"Message: {data['response']['message']}", file=sys.stderr) if data["request"]["status_code"] >= 400: sys.exit(1) @@ -386,14 +389,14 @@ def app(): if action is None: action = "help" else: - print("Multiple actions specified") + print("Multiple actions specified", file=sys.stderr) sys.exit(1) case "--list" | "-l": if action is None: action = "list" else: - print("Multiple actions specified") + print("Multiple actions specified", file=sys.stderr) sys.exit(1) case "--listed" | "-L" | "--public" | "-P": @@ -403,7 +406,7 @@ def app(): if action is None: action = "paste" else: - print("Multiple actions specified") + print("Multiple actions specified", file=sys.stderr) sys.exit(1) case "--file" | "-f": @@ -413,7 +416,7 @@ def app(): content = f.read() to_pass = 1 else: - print("Multiple actions specified") + print("Multiple actions specified", file=sys.stderr) sys.exit(1) case "--show" | "-s": @@ -441,14 +444,14 @@ def app(): if action is None: action = "delete" else: - print("Multiple actions specified") + print("Multiple actions specified", file=sys.stderr) sys.exit(1) case "--edit" | "-e": if action is None: action = "edit" else: - print("Multiple actions specified") + print("Multiple actions specified", file=sys.stderr) sys.exit(1) case "--debug": |