[Catalyst] Contributing code

Alexander Hartmaier alexander.hartmaier at t-systems.at
Wed Jun 30 12:40:51 GMT 2010


I'd prefer *much* more inline comments as well to make it easier to work
on code parts I've never touched before when debugging or adding
features as it's very hard to figure out what the methods do and how
they interact.

--
Best regards, Alex


Am Montag, den 21.06.2010, 16:55 +0200 schrieb J. Shirley:
> On Mon, Jun 21, 2010 at 7:43 AM, iain <iainhubbard at googlemail.com> wrote:
> > Ævar Arnfjörð Bjarmason wrote:
> >>
> >> It depends on what sort of comments you're making. Comments that
> >> explain tricky code that help with maintenance down the road are
> >> welcome everywhere. If you're just making comments that help someone
> >> completely unfamiliar with Catalyst to read the code it'll probably be
> >> more distracting than helpful to core devs.
> >>
> >>
> >
> > Surely the core devs can work with comments?
> >
> > Would having more comments not encourage more developers to contribute by
> > lowering the barrier for entry?
> >
>
> I don't think I can count as a core developer anymore, but really the
> more comments the better (provided they aren't there so you can
> practice your buddy career as a novelist).
>
> If you can't write sane comments, though, I don't think you can write
> sane code :)
>
> _______________________________________________
> List: Catalyst at lists.scsys.co.uk
> Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst
> Searchable archive: http://www.mail-archive.com/catalyst@lists.scsys.co.uk/
> Dev site: http://dev.catalyst.perl.org/


*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*
T-Systems Austria GesmbH   Rennweg 97-99, 1030 Wien
Handelsgericht Wien, FN 79340b
*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*
Notice: This e-mail contains information that is confidential and may be privileged.
If you are not the intended recipient, please notify the sender and then
delete this e-mail immediately.
*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*



More information about the Catalyst mailing list