Hi,

On Mon, Sep 30, 2013 at 3:29 PM, Slavik Goltser <[email protected]> wrote:

> Took a quick look yesterday, definitely going to incorporate some code.
>

Feel free, I don't have enough time to clean it up and push to CPAN… Two
comments on your code:

 * I would rename to Dancer::Template::Mason, given that the name of the
module is Mason, not Mason2.
 * Drop the data_dir part from your code: Mason defaults are safe enough,
no need to add different semantics at this layer, the user can always make
his own mind and change the config.

Bye,
-- 
Pedro Melo
@pedromelo
http://www.simplicidade.org/
xmpp:[email protected]
mailto:[email protected]
_______________________________________________
dancer-users mailing list
[email protected]
http://lists.preshweb.co.uk/mailman/listinfo/dancer-users

Reply via email to