Teach webkitbot and WKR how to respond to yt?
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 26 May 2013 02:57:51 +0000 (02:57 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 26 May 2013 02:57:51 +0000 (02:57 +0000)
https://bugs.webkit.org/show_bug.cgi?id=116775

Reviewed by Andreas Kling.

* Scripts/webkitpy/tool/bot/irc_command.py:
(Hi): Fixed the usage string.
(Hi.execute): Made it more generic so that it works in WKR.
(YouThere): Added.
(YouThere.execute): Added.
* Scripts/webkitpy/tool/commands/newcommitbot.py:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@150701 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/webkitpy/tool/bot/irc_command.py
Tools/Scripts/webkitpy/tool/commands/newcommitbot.py

index 2bfa10e..1fae309 100644 (file)
@@ -1,3 +1,17 @@
+2013-05-25  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Teach webkitbot and WKR how to respond to yt?
+        https://bugs.webkit.org/show_bug.cgi?id=116775
+
+        Reviewed by Andreas Kling.
+
+        * Scripts/webkitpy/tool/bot/irc_command.py:
+        (Hi): Fixed the usage string.
+        (Hi.execute): Made it more generic so that it works in WKR.
+        (YouThere): Added.
+        (YouThere.execute): Added.
+        * Scripts/webkitpy/tool/commands/newcommitbot.py:
+
 2013-05-25  Simon Fraser  <simon.fraser@apple.com>
 
         Try to fix the EFL build.
index 00da844..979fcff 100644 (file)
@@ -111,10 +111,10 @@ class Help(IRCCommand):
 
 class Hi(IRCCommand):
     usage_string = "hi"
-    help_string = "Retrieves a random quip from Bugzilla."
+    help_string = "Responds with hi."
 
     def execute(self, nick, args, tool, sheriff):
-        if len(args) and re.match(r'webkitbot_*\s*!\s*', ' '.join(args)):
+        if len(args) and re.match(sheriff.name() + r'_*\s*!\s*', ' '.join(args)):
             return "%s: hi %s!" % (nick, nick)
         quips = tool.bugs.quips()
         quips.append('"Only you can prevent forest fires." -- Smokey the Bear')
@@ -129,6 +129,14 @@ class PingPong(IRCCommand):
         return nick + ": pong"
 
 
+class YouThere(IRCCommand):
+    usage_string = "yt?"
+    help_string = "Responds with yes."
+
+    def execute(self, nick, args, tool, sheriff):
+        return "%s: yes" % nick
+
+
 class Restart(IRCCommand):
     usage_string = "restart"
     help_string = "Restarts sherrifbot.  Will update its WebKit checkout, and re-join the channel momentarily."
@@ -298,6 +306,7 @@ visible_commands = {
     "roll-chromium-deps": RollChromiumDEPS,
     "rollout": Rollout,
     "whois": Whois,
+    "yt?": YouThere,
 }
 
 # Add revert as an "easter egg" command. Why?
index b65b0c3..b098cb3 100644 (file)
@@ -37,6 +37,7 @@ from webkitpy.tool.bot.irc_command import Help
 from webkitpy.tool.bot.irc_command import Hi
 from webkitpy.tool.bot.irc_command import PingPong
 from webkitpy.tool.bot.irc_command import Restart
+from webkitpy.tool.bot.irc_command import YouThere
 from webkitpy.tool.bot.ircbot import IRCBot
 from webkitpy.tool.commands.queues import AbstractQueue
 from webkitpy.tool.commands.stepsequence import StepSequenceErrorHandler
@@ -52,6 +53,7 @@ class NewCommitBot(AbstractQueue, StepSequenceErrorHandler):
         "hi": Hi,
         "ping": PingPong,
         "restart": Restart,
+        "yt?": YouThere,
     }
 
     _maximum_number_of_revisions_to_avoid_spamming_irc = 10