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 +5
1 changed files with 5 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit b50a6aef00 - Show all commits

5
handlers.py
View file

@ -172,6 +172,10 @@ def PART(bot: bare.bot, msg: str) -> tuple[None, None]:
bot.channels.pop(channel, None) bot.channels.pop(channel, None)
return None, None return None, None
def QUIT(bot: bare.bot, msg: str) -> tuple[None, None]:
if bot.server == "replirc":
if msg.split("!", 1)[0][1:] == "FireMCBot":
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
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
@ -186,4 +190,5 @@ handles: dict[
"PART": PART, "PART": PART,
"MODE": NULL, "MODE": NULL,
"TOPIC": NULL, "TOPIC": NULL,
"QUIT": QUIT
} }