[GitHub] andscoop commented on issue #3699: [AIRFLOW-2853] Add official committers to README

2018-08-05 Thread GitBox
andscoop commented on issue #3699: [AIRFLOW-2853] Add official committers to 
README
URL: 
https://github.com/apache/incubator-airflow/pull/3699#issuecomment-410553138
 
 
   @r39132 Looks like a couple of these links need [team access to 
github](https://github.com/orgs/apache/teams/airflow-committers/members) and 
access to [whimsy server](https://whimsy.apache.org/roster/ppmc/airflow). So we 
can't put those in the file. The best definitive resource I have seen so far is 
the one @ashb [linked to in 
review](https://people.apache.org/committers-by-project.html#airflow). Is this 
the link we should move forward with?
   
   
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] andscoop commented on issue #3699: [AIRFLOW-2853] Add official committers to README

2018-08-05 Thread GitBox
andscoop commented on issue #3699: [AIRFLOW-2853] Add official committers to 
README
URL: 
https://github.com/apache/incubator-airflow/pull/3699#issuecomment-410549554
 
 
   @r39132 please try again - just officially signed up for the cwiki which is 
apparently different than the jira signup.
   
   I understand the concern with maintaining this is 3 different places. Main 
reason for the PR is IMO that docs which don't live with the code get left 
behind. I will gladly update the wiki if we choose not to move forward with the 
PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] andscoop commented on issue #3699: [AIRFLOW-2853] Add official committers to README

2018-08-05 Thread GitBox
andscoop commented on issue #3699: [AIRFLOW-2853] Add official committers to 
README
URL: 
https://github.com/apache/incubator-airflow/pull/3699#issuecomment-410545427
 
 
   @r39132 I did not originally research thoroughly. 
   
   It would require us to set owners to various sections of the codebase so 
that those owners are prompted for automatic review when a PR for that section 
is submitted. CODEOWNERS may be the direction the project wants to go 
eventually - but I don't believe it (the project) currently goes into the level 
of detail of accepting committers for specific sections of the codebase. While 
some committers may be more inclined to review PRs from a specific section, 
from the outside looking in, it appears that most committers/maintainers float 
across various sections of the codebase.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services