Pulled all the glossary and ability word stuff into the API, since it was uhhh already there anyway?
Regarding test changes:
def CLOWNS actually doesn't return an empty string right now, because it fuzzy matches to Conspire. It turns out that a lot of dumb searches fuzzy match, but this isn't a terribly abusable command (and if someone's spamming a channel with it, I'll happily kick them). If you care about this, I'm using fast-fuzzy to find matches, but the threshold needs to be like .5 to match all the various ways we should expect people to want to define roll a d20 (eg roll dice, roll a die etc)
All ability words are strictly lower-cased, so modified strive and will of the council there.
My glossary entries split up "Banding" and "Bands with Other," so modified that test to accommodate new response.
Fryyyyy
commented
3 years ago
Pulled all the glossary and ability word stuff into the API, since it was uhhh already there anyway?
Regarding test changes:
def CLOWNSactually doesn't return an empty string right now, because it fuzzy matches to Conspire. It turns out that a lot of dumb searches fuzzy match, but this isn't a terribly abusable command (and if someone's spamming a channel with it, I'll happily kick them). If you care about this, I'm using fast-fuzzy to find matches, but the threshold needs to be like .5 to match all the various ways we should expect people to want to defineroll a d20(egroll dice,roll a dieetc)All ability words are strictly lower-cased, so modified strive and will of the council there.
My glossary entries split up "Banding" and "Bands with Other," so modified that test to accommodate new response.