Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update arrow-site governance page #383

Closed
pitrou opened this issue Jul 17, 2023 · 10 comments · Fixed by #387
Closed

Update arrow-site governance page #383

pitrou opened this issue Jul 17, 2023 · 10 comments · Fixed by #387
Assignees

Comments

@pitrou
Copy link
Member

pitrou commented Jul 17, 2023

The check_committers list indicates that our governance page is out of date:

$ ../.venv/bin/python check_committers.py 
Missing PMCs in local list: ['jakevin']
Missing committers in local list: ['akurmustafa', 'avantgardner', 'gangwu', 'kevingurney', 'wayne']
Unexpected members in local list: ['avantgardnerio', 'jackwener', 'mustafasrepo', 'waynexia']
@pitrou
Copy link
Member Author

pitrou commented Jul 17, 2023

@alamb @raulcd

@kou
Copy link
Member

kou commented Jul 18, 2023

@kevingurney Do you want to update https://github.com/apache/arrow-site/blob/main/_data/committers.yml ?

@alamb
Copy link
Contributor

alamb commented Jul 18, 2023

@kevingurney here is an example of how to do so: #356

@alamb
Copy link
Contributor

alamb commented Jul 18, 2023

The check_committers list indicates that our governance page is out of date:

Missing PMCs in local list: ['jakevin']
Missing committers in local list: ['akurmustafa', 'avantgardner', 'gangwu', 'kevingurney', 'wayne']
Unexpected members in local list: ['avantgardnerio', 'jackwener', 'mustafasrepo', 'waynexia']

Some of these are due to members that have different github names than apache names and the github names being what is listed on the committer list.

For example akurmustafa is mustafasrepo,

If you have already scripted a check, perhaps we can make the script update the data automatically?

@pitrou
Copy link
Member Author

pitrou commented Jul 18, 2023

Some of these are due to members that have different github names than apache names and the github names being what is listed on the committer list.

Hmm, the current convention is to put ASF IDs in the committer list, AFAIR.

If you have already scripted a check, perhaps we can make the script update the data automatically?

How would it do that? I think it's simpler to have people fill in the right information :-)

@kevingurney
Copy link
Member

Thank you @kou and @alamb! I will follow up with a PR to add myself to committers.yml.

@kevingurney
Copy link
Member

Follow up PR: #385

@alamb
Copy link
Contributor

alamb commented Jul 19, 2023

How would it do that? I think it's simpler to have people fill in the right information :-)

The challenge is that you have to get humans to do something, which is often more challenging than getting computers to do the same thing, I have found.

I was imagining the script could update the contents of data/committers.yml locally

Someone would still have to run it manually and make a PR but that is probably easier than keeping the contents up to date manually.

If I get a chance I will play around with it

@alamb alamb self-assigned this Jul 24, 2023
@alamb
Copy link
Contributor

alamb commented Jul 24, 2023

I plan to work on this issue this week

@alamb alamb changed the title Update governance page Update arrow-site governance page Jul 24, 2023
@alamb
Copy link
Contributor

alamb commented Aug 7, 2023

Here is one PR to fix incorrect aliases: #387

Here is a second PR to add @wgtmac #388

alamb added a commit that referenced this issue Aug 7, 2023
Related to #383

Add Gang Wu to committers list per
https://lists.apache.org/thread/sbc7kyjgv3bwptlydr8wspnd6zbzptvv

@wgtmac  what affiliation would be good to list for you?
@alamb alamb closed this as completed in #387 Aug 8, 2023
alamb added a commit that referenced this issue Aug 8, 2023
…rmustafa` and `wayne` (#387)

Closes #383

# Rationale

The committers page has links to the apache phonebook and thus the alias
should be the apache user name (rather than github username):
https://arrow.apache.org/committers/

@pitrou noticed that there were some entries that used the github names
instead

# Changes

Correct committer data for `jakevin`, `avantgardner`, `akurmustafa` and
`wayne`


Note: I will make a separate PR to add an entry for Gang Wu


cc @waynexia @avantgardnerio @mustafasrepo @jackwener
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants