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/