ServiceNowDevProgram / SlackerBot

The official application repository for the bot @Slacker on the sndevs.com workspace.
https://github.com/ServiceNowDevProgram/Hacktoberfest
15 stars 124 forks source link

Discussion - change default behavior to assume that regex should be bounded #337

Open wiz0floyd opened 1 month ago

wiz0floyd commented 1 month ago

Current behavior: mid-word matches trigger related parsers.

Suggested behavior: implicit /b at the beginning and end for all matching and add a checkbox to the parser to explicitly indicate when mid word matching is intended behavior.

eriemer1 commented 1 month ago

Makes sense to me. We could also have an optional flag for the parser to explicitly be not bounded when that makes sense, like the @chuck.tomasi one

wiz0floyd commented 1 month ago

Alternatively we could have implicit white space on the ends so that mid sentence still triggers but not mid word.