basic: Added a reprocess command

mrshu / brutal-plugins

A set of plugins for brutal, the mighty chatbot

Apache License 2.0

4 stars 2 forks source link

basic: Added a reprocess command #76

Closed mrshu closed 9 years ago

mrshu commented 9 years ago

Added a reprocess command which kind of 'repeats' the last command or a message of a user to a given bot.

Signed-off-by: mr.Shu mr@shu.io

mrshu commented 9 years ago

to @pepol @Adman

I couldn't find a better name than this. reprocess seems artificial (and lacks the "what" part), repeat has the same problem so r it is.

pepol commented 9 years ago

@mrshu what about redo?

mrshu commented 9 years ago

@pepol yeah, that might work. However, I'd still like to keep the the short r.

pepol commented 9 years ago

Sure, it was just an idea (or maybe even . as in vim command repeat could work? :-D) otherwise LGTM!

mrshu commented 9 years ago

@pepol that would imply you are in normal mode =)

pepol commented 9 years ago

Well one more thing though. Do we really want to use ''' for docstrings? I mean there are some (older) functions using """ in this file and the new ones using '''. Also, IIRC, it is preferred to use """ for docstrings (but this may be Google-specific, not PEP8, I'm not sure here).

mrshu commented 9 years ago

@pepol good catch!

That was some sort of a misunderstanding between my two selves in different points in time.

What I plan to do with this is to leave it as it is and create an "easy" bug for someone who wants to get acquainted with the codebase to consolidate these docstring brackets in this file.