Skip to content

MINOR: [Website] Add Alenka, Dewey and Rok to commiters#221

Merged
kou merged 4 commits intoapache:masterfrom
rok:website-new-commiters
Jul 6, 2022
Merged

MINOR: [Website] Add Alenka, Dewey and Rok to commiters#221
kou merged 4 commits intoapache:masterfrom
rok:website-new-commiters

Conversation

@rok
Copy link
Member

@rok rok commented Jul 5, 2022

No description provided.

@github-actions
Copy link

github-actions bot commented Jul 5, 2022

Thanks for opening a pull request!

Could you open an issue for this pull request on JIRA?
https://issues.apache.org/jira/browse/ARROW

Then could you also rename pull request title in the following format?

ARROW-${JIRA_ID}: [${COMPONENT}] ${SUMMARY}

See also:

@rok
Copy link
Member Author

rok commented Jul 5, 2022

@AlenkaF @paleolimbot did I get your details right?

@pitrou
Copy link
Member

pitrou commented Jul 5, 2022

Did you run scripts/check_committers.py (I'm not sure it's run on CI)?

@rok
Copy link
Member Author

rok commented Jul 5, 2022

Did you run scripts/check_committers.py (I'm not sure it's run on CI)?

I had not. Good call:

Missing PMCs in local list: ['houqp']
Missing committers in local list: ['alenka', 'liukun', 'quinnj']
Unexpected members in local list: ['alenkaf', 'liukun4515']

@pitrou
Copy link
Member

pitrou commented Jul 5, 2022

Ah, it looks like alenka wins against alenkaf.

@rok
Copy link
Member Author

rok commented Jul 5, 2022

@pitrou that is just the Apache phonebook!

Comment on lines +231 to +234
- name: Jacob Quinn
role: Committer
alias: quinnj
affiliation: RelationalAI
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@quinnj is this right?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct!

@paleolimbot
Copy link
Member

Thanks! Looks good to me!

@kou kou merged commit 45ac797 into apache:master Jul 6, 2022
@rok rok deleted the website-new-commiters branch July 7, 2022 08:29
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 this pull request may close these issues.

7 participants