Anthony Gladdish wrote:>
I would like to know if I've gone about it correctly? And if so, could you 
review my patch and possibly apply?

This patch looks fine, except:

. You don't need to explicitly call Class::C3::initialize
. You need to add the MRO::Compat dep to Makefile.PL
. Add what you did to the Changelog, and your name to the contributors in the POD if you want to. :)

Easiest way to get your patch into the dist is to fork:

http://github.com/hdp/catalyst-action-rest/tree/master

apply, then send a pull request.

Well volunteered!

Cheers
t0m


_______________________________________________
List: [email protected]
Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst
Searchable archive: http://www.mail-archive.com/[email protected]/
Dev site: http://dev.catalyst.perl.org/

Reply via email to