forked from Charcoal-SE/SmokeDetector
-
Notifications
You must be signed in to change notification settings - Fork 0
more user‐friendly handling of chat commands
Jeff Schaller edited this page Apr 27, 2024
·
25 revisions
- Come up with a proposal (all-inclusive?) and post it as an Issue on the CHQ repo: https://github.com/Charcoal-SE/SmokeDetector/issues
- discuss which to include and exclude
- create PR with the consensus
Items:
- make the split more forgiving in the "report" chat command (https://github.com/Charcoal-SE/SmokeDetector/pull/10755): change
urls = argsraw[0].split(' ')
tourls = argsraw[0].split(sep=None)
- !!/reject 12345 5 false positives, no known hits.; account for \d+ (19+ characters)
- maybe make exceptions for < 19 chars if: 'superseded by \d+' or match('author') && match('request')
- for watches, allow leading whitespace (if that's the only problem)
- addblu -- allow a trailing comment (throw away the comment?)