Add aliases for the "sheriffs" command that cover the common misspellings that people make in sheriffbot. Misspellings: sherrifs, sherriffs, sherifs
Created attachment 182503 [details] Patch
GTLM :) Eric would say, "We might want to test this, but OK". Let's wait for his OK.
Just saw https:bug 106754
Ahem, bug 106754 so this LGTM.
Comment on attachment 182503 [details] Patch This is possibly the best patch I've seen this year. I can't spell sherrrifs to save my life.
Comment on attachment 182503 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=182503&action=review > Tools/Scripts/webkitpy/tool/bot/irc_command.py:299 > +# Enough people misspell "sheriffs" that they've requested aliases for the command. > +commands["sherriffs"] = Sheriffs > +commands["sherifs"] = Sheriffs > +commands["sherrifs"] = Sheriffs This is also just a crutch for a "did you mean" behavior, but that's fine. We'll do that in SB 2.0 :)
Comment on attachment 182503 [details] Patch Clearing flags on attachment: 182503 Committed r139585: <http://trac.webkit.org/changeset/139585>
All reviewed patches have been landed. Closing bug.
(In reply to comment #5) > (From update of attachment 182503 [details]) > This is possibly the best patch I've seen this year. I can't spell sherrrifs to save my life. Hahaha. I'm very glad. (: