[GitHub] [accumulo-website] keith-turner commented on a diff in pull request #380: Fixes #379 creates emeritus PMC table on people page

["keith-turner (via GitHub)" <gi...@apache.org>]

keith-turner commented on code in PR #380:
URL: https://github.com/apache/accumulo-website/pull/380#discussion_r1154829388


##########
pages/people.md:
##########
@@ -62,6 +60,14 @@ $(function() {
 | vikrams       | Vikram Srivastava                                 | [Cloudera][CLOUDERA]                   | [PT][PT] |
 | vines         | John Vines                                        | [sqrrl][SQRRL]                         | [ET][ET] |
 
+## Emeritus PMC and Committers

Review Comment:
   > I'm worried that this implies "Emeritus PMC" and "Emeritus Committers"
   
   I agree, tt can definitely read that way.
   
   > I can't think of a good way to communicate that succinctly
   
   Looking at what you suggested, I like "Committers Only (PMC Emeritus)" and its only a few bytes longer so it still succinct. 
   
   > but then what if we have somebody who accepts an invite as committer, but declines being PMC
   
   We can worry about that case in the future when it happens.  We could create another section or maybe add a column to the table. 
   
   
   



-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org