search

oasp / oasp4j

The Open Application Standard Platform for Java
Apache License 2.0
60 stars 303 forks source link

Remove @author tags #455

Closed maybeec closed 6 years ago

maybeec commented 8 years ago

to be discussed:

I would like to remove the @author tags especially in the code templates as IMHO they are simply obsolete. Some guys are simply starting maintaining the tags, some are not... We already have version control in place and can blame someone if we like :D so why having the @author tags?

amarinso commented 8 years ago

In fact it is redundant information and noise for the audience

De:Malte Brunnlieb notifications@github.com Enviado:27 jul. 2016 2:10 p. m. Para:oasp/oasp4j Asunto: [oasp/oasp4j] Remove @author tags (#455)

to be discussed:

I would like to remove the @author tags especially in the code templates as IMHO they are simply obsolete. Some guys are simply starting maintaining the tags, some are not... We already have version control in place and can blame someone if we like :D so why having the @author tags?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/oasp/oasp4j/issues/455, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AI6zpyRR88ReUJw_kIrWbOk66uit9diXks5qZ0qAgaJpZM4JWHzO. This message contains information that may be privileged or confidential and is the property of the Capgemini Group. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain, copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message.

hohwille commented 8 years ago

For cobigen you could consider to support variable expression such as eclipse does as static @author names in a template do not make any sense. However, I agree with you that we could also remove all @author tags. If someone wants to know the author he could dig in git history and see more of the truth than in any @author tag.

hohwille commented 6 years ago

PR was merged so done.