On 18/07/2013, at 11:27 AM, Rene Groeschke <gra...@breskeby.com> wrote:
> Luke Daley wrote: >> Hi, >> >> I'd like to propose that we instate a rule of not allowing any kind of >> author info in the codebase. >> >> I don't think it adds any value, and it gets awkward when refactoring >> contributed code with author attributions. At what point is it right to >> remove it? By never having it in there we don't have that problem. >> >> I'd like to: >> >> 1. Remove all the @author tags and similar >> 2. Add a checkstyle rule to error on @author tags >> 3. Update CONTRIBUTING.md to ask people not to submit pull requests with it >> on >> > > Sounds good to me. I'm not sure if #3 is necessary, as it is > automatically checked by the checkstyle rule and we don't apply failing > code anyway. I think it's useful as it sets expectations. That is, people don't get pissed off when we remove their author tags when we merge their pull request. -- Luke Daley Principal Engineer, Gradleware http://gradleware.com --------------------------------------------------------------------- To unsubscribe from this list, please visit: http://xircles.codehaus.org/manage_email
