From f0119faacfd9db6f972a56d74e327a00a2e29a48 Mon Sep 17 00:00:00 2001 From: Mohammad Ibrahim <74553450+Ibrahim2750mi@users.noreply.github.com> Date: Fri, 3 Mar 2023 01:53:55 +0530 Subject: Wookie's review + Double quotes for consistency + Using results["stdout"] instead of output so the warning is still displayed if the error doesn't fit in the message. + Using endswith so we don't need to hardcode the constant. Co-authored-by: wookie184 --- bot/exts/utils/snekbox.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bot/exts/utils/snekbox.py b/bot/exts/utils/snekbox.py index 7ee5c5d21..1c0896804 100644 --- a/bot/exts/utils/snekbox.py +++ b/bot/exts/utils/snekbox.py @@ -347,7 +347,7 @@ class Snekbox(Cog): warning_message = "" # 33 is the length of the error message. It is done to make sure the last line of output contains # the error and the error is not manually printed by the author with a syntax error. - if "EOFError: EOF when reading a line" in output[-33:] and results['returncode'] == 1: + if results["stdout"].rstrip().endswith("EOFError: EOF when reading a line") and results["returncode"] == 1: warning_message += ":warning: Note: `input` is not supported by the bot :warning:\n\n" icon = self.get_status_emoji(results) -- cgit v1.2.3