search

idkclub / dicebot

🎲 /roll support for Slack
https://dicebot.idk.club
The Unlicense
53 stars 20 forks source link

Fix dice roller syntax to be case-insensitive. #8

Closed katre closed 8 years ago

katre commented 8 years ago

Fixes #7.

arkie commented 8 years ago

Feel like using [dD] would be clearer than setting the entire regex to be case insensitive.

Otherwise seems good.

katre commented 8 years ago

I initially just changed the regexp to [dD], but since this is input that is being entered by the user, none of it should be case-sensitive. Adding specific case-sensitive syntax to this will end up just confusing people down the line.

arkie commented 8 years ago

Realized I also missed the [kK] in the regex so yeah, that makes sense. 👍