[GitHub] andscoop commented on issue #3699: [AIRFLOW-2853] Add official committers to README
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
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
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