[TYPO3-team-core-v5] JavaScript CGL Draft for Phoenix
Ernesto Baschny [cron IT]
ernst at cron-it.de
Fri Oct 8 09:25:44 CEST 2010
Karsten Dambekalns schrieb am 04.10.2010 20:49:
>> @author should stay, same as php.
>
> We think of dropping it. If needed, [scm] blame|praise can tell you in more detail. Plus @author gets out of sync anyway - or - if you add it for *every* change, it gets out of hand.
offtopic, but I agree with Steffen. "blame" is technically useful but
does not really honour the real "author" of a class. If you do a CGL
cleanup or refactor some methods, your name will be on almost every
line, but you are still not the author.
So having the "original author" in the top of the file is the most
useful way of honour the "main author". As Steffen said, if someone
contributes a major enhancement to a class, he can add himself to the
"list". If you add only a few lines, or fix a bug, you don't. Its leaves
it open to the authors to decide.
It also provides some feedback to people reading a class "in case of
questions, I might consider talking to @author".
Sometimes in v4 I open some class I've never seen before and looking at
the @author I often say "oh, Person XY did that, nice to know that he's
an expert on that area .."
Cheers,
Ernesto
More information about the TYPO3-team-core-v5
mailing list