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

2
bot.py
View file

@ -270,7 +270,7 @@ class bot(bare.bot):
tdict = {} tdict = {}
for thread in self.threads: for thread in self.threads:
tdict[thread] = timers.data[thread] tdict[thread] = timers.data[thread]
if thread in ["radio"]: if tdict[thread]["passInstance"]:
Firepup650 marked this conversation as resolved Outdated
Firepup650 commented 2024-05-01 01:03:14 +00:00
Outdated
Review
Copy link

Critical change, needs to be un-reverted

Critical change, needs to be un-reverted
9pfs marked this conversation as resolved Outdated
Firepup650 commented 2024-05-01 01:09:01 +00:00
Outdated
Review
Copy link

This needs to be indented another layer

This needs to be indented another layer
9pfs marked this conversation as resolved Outdated
Firepup650 commented 2024-05-01 01:13:47 +00:00
Outdated
Review
Copy link

We CANNOT use a tab here, this will crash the bot. It absolutely must be 4 spaces.

We CANNOT use a tab here, this will crash the bot. It absolutely must be 4 spaces.
9pfs commented 2024-05-01 01:15:12 +00:00
Outdated
Review
Copy link

Uh, what? I thought that WAS 4 spaces!

Uh, what? I thought that WAS 4 spaces!
😕 1
tdict[thread]["args"] = [self] tdict[thread]["args"] = [self]
tMgr = Thread(target=timers.threadManager, args=(tdict,)) tMgr = Thread(target=timers.threadManager, args=(tdict,))
tMgr.daemon = True tMgr.daemon = True

2
config.py
View file

@ -6,7 +6,7 @@ from typing import Optional, Any
import bare, pylast import bare, pylast
load_dotenv() load_dotenv()
__version__ = "v3.0.6" __version__ = "v3.0.7-dev"
npbase: str = ( npbase: str = (
"\[\x0303last\.fm\x03\] [A-Za-z0-9_[\]{}\\|\-^]{1,$MAX} (is listening|last listened) to: \x02.+ - .*\x02( \([0-9]+ plays\)( \[.*\])?)?" # pyright: ignore [reportInvalidStringEscapeSequence] "\[\x0303last\.fm\x03\] [A-Za-z0-9_[\]{}\\|\-^]{1,$MAX} (is listening|last listened) to: \x02.+ - .*\x02( \([0-9]+ plays\)( \[.*\])?)?" # pyright: ignore [reportInvalidStringEscapeSequence]
) )

11
timers.py
View file

@ -112,11 +112,18 @@ def radio(instance: bare.bot) -> NoReturn:
instance.log("Thread while loop broken", "FATAL") instance.log("Thread while loop broken", "FATAL")
exit(1) exit(1)
def mcDown(instance: bare.bot) -> None: def mcDown(instance: bare.bot) -> None:
Firepup650 marked this conversation as resolved Outdated
Firepup650 commented 2024-05-01 01:08:57 +00:00
Outdated
Review
Copy link

radio needs to use "passInstance": True, not "args": []

`radio` needs to use `"passInstance": True`, not `"args": []`
instance.sendraw("TOPIC #firemc :FireMC Relay channel (offline)") instance.sendraw("TOPIC #firemc :FireMC Relay channel (offline)")
data: dict[str, dict[str, Any]] = { data: dict[str, dict[str, Any]] = {
"radio": {"noWrap": True, "func": radio, "args": []}, "radio": {"noWrap": True, "func": radio, "passInstance": True},
Firepup650 marked this conversation as resolved Outdated
Firepup650 commented 2024-05-01 01:03:58 +00:00
Outdated
Review
Copy link

Reverting that commit didn't even remove the timer, lol

Reverting that commit didn't even remove the timer, lol
9pfs commented 2024-05-01 01:04:36 +00:00
Outdated
Review
Copy link

Looking for other commits to revert then

Looking for other commits to revert then
Firepup650 commented 2024-05-01 01:05:45 +00:00
Outdated
Review
Copy link

Just revert the revert, I'll delete the stuff that needs removal myself if you want.

Just revert the revert, I'll delete the stuff that needs removal myself if you want.
"mc-down": {"noWrap": False, "func": mcDown, "args": [], "interval": 60, "ignoreErrors": True} "mc-down": {
Firepup650 marked this conversation as resolved Outdated
Firepup650 commented 2024-05-01 01:03:37 +00:00
Outdated
Review
Copy link

related to my other commit, this needs to use the new passInstance param.

related to my other commit, this needs to use the new `passInstance` param.
"noWrap": False,
"func": mcDown,
"passInstance": True,
"interval": 60,
"ignoreErrors": True,
},
} }