idkclub / dicebot

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

parsing words with number #29

Closed whbdupree closed 7 years ago

whbdupree commented 7 years ago

Is this intentional behavior? Here, I have not written 'for test1' by omitting the 'for'. /roll d6+6 test1 and dicebot gives me @xxxx rolled 6 + 6 + 1 = 13

Dicebot has parsed the string 'test1' and interpreted it as intent to add 1.

arkie commented 7 years ago

The parser is fairly lax, but I'd consider this more undefined behavior rather than intentional. It doesn't default to the for style labelling since that would result in it possibly missing additional rolls or modifications.