Style Guide

References:
try to use simple and understandable paths when referencing some service/plugin. self.root is better than self.parent.parent.parent. self.parent is better than self.root.getNamedService('someservice').bla if the service is the parent.

Logging:
use logger.error ONLY for messages, which should be reported as bugs. Exceptions, “this should never happen” conditions1), etc.

Testing

unit-tests are great, doc-tests are good. but most functions can only be human-tested because of the nature of an IRC bot.
if you tested your plugin successfully, make an entry at the tests page, so we know which reversion worked, and that it was successfully tested.