From ae6f8c5550ca8e4bf58ecf0bbfecc8a9dcbcd43d Mon Sep 17 00:00:00 2001 From: ache Date: Fri, 23 Jan 2026 13:37:10 +0100 Subject: 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. --- src/lolbin/cli.py | 19 +++++++++++-------- 1 file 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 ") + print( + "Configure lolbin ", + 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": -- cgit v1.3-2-g11bf