Nice. Thx Kaxil. -s On Sun, Aug 5, 2018 at 3:57 PM Naik Kaxil <k.n...@reply.com> wrote:
> Last time I updated to add my name, it was reflected the next day. > > I had added the instructions here when I joined PPMC: > https://cwiki.apache.org/confluence/display/AIRFLOW/Committers%27+Guide#Committers'Guide-AddingNewCommitters/PMCMemberstotheAirflowIncubationStatusPage > > Jenkins job runs daily at around 01:39 AM for the entire incubator site: > https://builds.apache.org/view/H-L/view/Incubator/job/Incubator%20Site/ > > Regards, > Kaxil > > On 05/08/2018, 21:44, "Sid Anand" <san...@apache.org> wrote: > > What are the instructions to add committers to this page? > -s > > To update http://incubator.apache.org/projects/airflow.html, you need > to > check out the incubator SVN repo and modifying > content/projects/airflow.xml > > I SVN'committed the change below for tfeng but am not sure when the > html > page gets rebuilt and redeployed to the site. Some CI/CD process I > believe. > > sianand@LM-SJN-21002367:~/Projects/incubator/content/projects $ svn > diff > > Index: airflow.xml > > =================================================================== > > --- airflow.xml (revision 1837468) > > +++ airflow.xml (working copy) > > @@ -180,6 +180,11 @@ > > <td>Kaxil Naik</td> > > </tr> > > <tr> > > + <td>.</td> > > + <td>tfeng</td> > > + <td>Tao Feng</td> > > + </tr> > > + <tr> > > <td>Extra</td> > > <td>.</td> > > <td>.</td> > > Every time we add a committer, we need to update this page. Here's some > info on the SVN repo so you can check it out for future reference: > > sianand@LM-SJN-21002367:~/Projects/incubator/content/projects $ svn > info > > Path: . > > Working Copy Root Path: /Users/sianand/Projects/incubator > > URL: > > https://svn.apache.org/repos/asf/incubator/public/trunk/content/projects > > Relative URL: ^/incubator/public/trunk/content/projects > > Repository Root: https://svn.apache.org/repos/asf > > Repository UUID: 13f79535-47bb-0310-9956-ffa450edef68 > > Revision: 1837468 > > Node Kind: directory > > Schedule: normal > > Last Changed Author: mck > > Last Changed Rev: 1837396 > > Last Changed Date: 2018-08-03 19:07:54 -0700 (Fri, 03 Aug 2018) > > On Sun, Aug 5, 2018 at 1:39 PM Ash Berlin-Taylor < > ash_airflowl...@firemirror.com> wrote: > > > - https://github.com/orgs/apache/teams/airflow-committers/members > < > > https://github.com/orgs/apache/teams/airflow-committers/members> > > > > This one at least is populated automatically via a 30-minutely cron > job. > > > > > > > On 5 Aug 2018, at 21:35, Sid Anand <san...@apache.org> wrote: > > > > > > Committers/Mentors, > > > We have several places where committers are listed: > > > > > > - https://whimsy.apache.org/roster/ppmc/airflow > > > - http://incubator.apache.org/projects/airflow.html > > > - Mentors, is the committer list here meant to be current? > > > - Kaxil, I added tfeng to this page (done via SVN).. though > I'm not > > > sure when the actual publishing happens to this site. I > believe > > it's done > > > via some CI/CD process now. > > > - > https://github.com/orgs/apache/teams/airflow-committers/members > > > - This is needed for the Gitbox integration we are using now. > Does it > > > get populated automatically from whimsy? > > > > > > When we promote a contributor to committer/PPMC, do we need to > update > > all 3 > > > of these places? > > > > > > FYI, there is a PR to also add this list to the README so the > community > > can > > > see who the committers are : > > > https://github.com/apache/incubator-airflow/pull/3699 > > > > > > Currently, the committer section of the README points to a wiki > page that > > > displays all of these links. > > > > > > And if you wanted more options, GitHub's CODEOWNERS file provides > some > > > interesting functionality, though I don't think we need it: > > > https://blog.github.com/2017-07-06-introducing-code-owners/ > > > > > > -s > > > > > > > > > > > Kaxil Naik > > Data Reply > 2nd Floor, Nova South > 160 Victoria Street, Westminster > London SW1E 5LB - UK > phone: +44 (0)20 7730 6000 > k.n...@reply.com > www.reply.com >
