lizmat / IRC-Client

Raku module for building IRC (Internet Relay Chat) clients
Artistic License 2.0
14 stars 7 forks source link

Make , : mandatory in irc-to-me (<[,:\s]> → <[,:]>) #21

Closed AlexDaniel closed 7 years ago

AlexDaniel commented 7 years ago

Here is the line in question:

 if $e.text.subst-mutate: /^ $nick <[,:\s]> \s* /, '' {

I propose to remove \s from the delimiters.

Bot::BasicBot has a similar regex, and a lot of people were complaining about it. While I'm totally OK with using the space, people constantly trigger the bot unintentionally when trying to talk about it.

For example:

<AlexDaniel> bisectable6 is a very nice bot!
<bisectable6> AlexDaniel, On both starting points (good=2015.12 bad=77724b2) the exit code is 1 and the output is identical as well
<bisectable6> AlexDaniel, Output on both points: ===SORRY!=== Error while compiling /tmp/l9JSf9Knsh␤Bogus postfix␤at /tmp/l9JSf9Knsh:1␤------> is a very nice bot<HERE>!␤    expecting any of:␤        infix␤        infix stopper␤        postfix␤        statement end␤        statement modifier␤        statement modifier loop

Or maybe it can be optional?

zoffixznet commented 7 years ago

Fixed and shipped in version 3.005001