Code review comment for lp:~andrea.corbellini/ubuntu-bots/i-am-only-a-bot-reply

Revision history for this message
m4v (m4v) wrote :

Andrea, thanks for helping us with Ubottu! :D

I agree with you in changing the way ubottu says the "I'm not smart" message, however, this patch only gets rid of the "Error: " bit of text, it doesn't prefix the user's nick like your description says, it will say "<ubottu> I am only a bot ..." If you want the bot to address the user, you have to use prefixNick=True, like irc.reply("I am only a bot ...", prefixNick=True)

Andrea, since your patch doesn't quite does what the description says, I'm not sure if I should approve, will you revise it? :) You can test it by setting up an ubottu clone, if you need help with that you can join #ubuntu-bots-devel on freenode and ask me or tsimpson.

« Back to merge proposal