Splyth / slack_bot

Slack Bot Created for the Animutiny Slack Group (Name is Lisa)
MIT License
9 stars 5 forks source link

#23: Adding roll me command for dice-rolling support #67

Closed jedibob5 closed 4 years ago

jedibob5 commented 4 years ago

Resolves #23

Dice-rolling command. Pretty straightforward. Syntax is roll me [number of dice]d[number of sides], i.e. roll me 3d8. Includes a alias to just roll for convenience's sake.

The feature request (#23) specified a maximum of 300d20, but d100s are fairly common in TTRPGs, plus the die size doesn't really have any performance impact, so I have it at a max of 300d1000 right now. I'm open to suggestions if this should be changed, though I tested it on the highest possible roll and the response time is still pretty quick.

Also fixed a couple of minor typos while I was at it, and added a little Easter egg, if that's acceptable. Don't tell anyone about it right away, I'd like to see if anyone stumbles onto it first.

jedibob5 commented 4 years ago

@Zylox @noah-guillory @Mill1893 Need another review on this when yall have some time