Chris, please see the current Email action in do/future for some style tips.

Also, please consider decomposing the Perform action into smaller, simpler 
functions. Local variables should be camel case (sendEmail, not send_email).
-- 
https://code.edge.launchpad.net/~cszikszoy/do-plugins/evolution-plugin/+merge/2033
You are subscribed to branch lp:do-plugins/future.

_______________________________________________
Mailing list: https://launchpad.net/~do-plugins
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~do-plugins
More help   : https://help.launchpad.net/ListHelp

Reply via email to