← Back to team overview

do-plugins team mailing list archive

Thread
Date

Re: Proposed merge of lp:~cszikszoy/do-plugins/evolution-plugin into lp:do-plugins/future

To: GNOME Do Plugins Team <do-plugins@xxxxxxxxxxxxxxxxxxx>
From: David Siegel <djsiegel@xxxxxxxxx>
Date: Tue, 09 Dec 2008 18:22:13 -0000
In-reply-to: <20081202052833.3843.42785.launchpad@potassium.ubuntu.com>
Reply-to: mp+2033@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

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.

References

Proposed merge of lp:~cszikszoy/do-plugins/evolution-plugin into lp:do-plugins/future
From: Chris S., 2008-12-02