Firepup650/FireBot
2
0
Fork 1
Code Issues 5 Pull requests Activity Actions

Avoid spamming #firemc #6

Closed
9pfs wants to merge 14 commits from 9pfs/FireBot:dont-spam-firemc into master
pull from: 9pfs/FireBot:dont-spam-firemc
merge into: Firepup650:master
Conversation 21 Commits 14 Files changed 3 +1
1 changed files with 1 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 7e927ff7fe - Show all commits

1
handlers.py
View file

@ -176,6 +176,7 @@ def QUIT(bot: bare.bot, msg: str) -> tuple[None, None]:
if bot.server == "replirc": if bot.server == "replirc":
if msg.split("!", 1)[0][1:] == "FireMCBot": if msg.split("!", 1)[0][1:] == "FireMCBot":
bot.send("TOPIC #firemc :FireMC Relay channel (offline)\n") bot.send("TOPIC #firemc :FireMC Relay channel (offline)\n")
Firepup650 marked this conversation as resolved
Firepup650 commented 2024-05-01 00:58:26 +00:00
Review
Copy link

Last line of the function MUST be return None, None, it'll crash otherwise.

Last line of the function _**MUST**_ be `return None, None`, it'll crash otherwise.
9pfs commented 2024-05-01 01:01:50 +00:00
Review
Copy link

Fixing

Fixing
return None, None
def NULL(bot: bare.bot, msg: str) -> tuple[None, None]: def NULL(bot: bare.bot, msg: str) -> tuple[None, None]:
return None, None return None, None