ctubbsii commented on a change in pull request #188: Blog post for upgrading to 
2.0
URL: https://github.com/apache/accumulo-website/pull/188#discussion_r313155339
 
 

 ##########
 File path: _posts/blog/2019-08-12-why-upgrade.md
 ##########
 @@ -0,0 +1,138 @@
+---
+title: "Top 10 Reasons to Upgrade"
+author: Mike Miller
+reviewers: Keith Turner, Christopher Tubbs
 
 Review comment:
   I don't want my name here. I also don't think any individual names should be 
here. I think individual names here are too similar to `@author` tags in code, 
promoting the idea of individuals over community, which is something that the 
ASF expressly decided against many years ago. The commit logs show the authors 
and the mailing list show reviewer feedback. That should be sufficient if 
somebody wants to go back and review that information. For the purpose of the 
presentation on the website, the content should reflect the community as a 
whole.
   
   Also, this "reviewers" line is just plain weird to me. It's likely that 
others reviewed the page, too, but didn't find have any suggestions for 
changes. Does their passive concurrence count for less?
   
   I don't know... perhaps these should be discussed on the dev@ list, and 
perhaps the reviewers tag can be addressed separately from the author tag. In 
either case, I'm not comfortable with my name being added here.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to