Results 296 comments of arielle

> I was able to find this, the limit is already 100 lines, so my point may be invalid. > > https://github.com/python-discord/bot/blob/a5cfe2ac47077805c205bcaf05eaed5ded5d09f9/config-default.yml#L439-L442 Depending on how the messages trigger it, it...

So it seems like there are still some modifications, and I do understand this a *bit* more, but I think that increasing the max newlines a bit *might* help in...

I'll be fixing this commit history tonight

As far as I can tell the mfa code is no longer necessary as discord doesn't have mfa tokens like that now.

There's also a bit of the matter that this regex is now out of date. I don't know what the current token length is.

The only inaccurate part is the last section is too short as it is, but because of how the parsing works, that isn't an issue, i suppose

I'd like to implement this. EDIT: Just noticed that @TizzySaurus is assigned, so *if* you don't want to implement it (since it been a month), I can. :D

I can implement this tonight, if no one else wants to do it before then

> Looks like this is caused by how we're loading extensions, if the source cog is loaded before the tags cog, then the `TagIdentifier` class used by source is different...

> While that does solve the problem, I feel like it's more of a bandaid solution. > A similar error is likely to occur for other extensions if they end...