• Move decorators to module. It really wasn't clear how decorators inside classes work, and why they didn't have a self parameter, but still wasn't static. This also makes them effectively private without having to underscore them, which I think we should have done long time ago atm. Note that this might break backwards compatibility slightly (only if people are daft enough to have used the decorators themselves)
• Don't call _message_wrapper directly. Ever. Instead always use the message decorator, since it's what it's there for. Closes Weirdness with Bot._message_wrapper #627
• Don't use the message decorator if the method isn't supposed to return a message. The decorator could handle values like True (which is often the return value), but to someone reading the code, it seems like it's a message returning method even when it wasn't.
• Always document timeout and **kwargs
• Log all methods

Mostly done to make pycharm stop bloody complaining :P