[Catalyst] Catlyst::Action::REST - NEXT warnings patch

Tomas Doran bobtfish at bobtfish.net
Sat May 9 17:02:13 GMT 2009


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




More information about the Catalyst mailing list