[GitHub] [kafka] vvcephei commented on a diff in pull request #13842: KAFKA-14995: Automate asf.yaml collaborators refresh

["vvcephei (via GitHub)" <gi...@apache.org>]

vvcephei commented on code in PR #13842:
URL: https://github.com/apache/kafka/pull/13842#discussion_r1239120109


##########
refresh-collaborators.py:
##########
@@ -0,0 +1,44 @@
+import os
+from bs4 import BeautifulSoup
+from github import Github
+import yaml
+
+### GET THE NAMES OF THE KAFKA COMMITTERS FROM THE apache/kafka-site REPO ###
+github_token = os.environ.get('GITHUB_TOKEN')
+g = Github(github_token)
+repo = g.get_repo("apache/kafka-site")
+contents = repo.get_contents("committers.html")
+content = contents.decoded_content
+soup = BeautifulSoup(content, "html.parser")
+committer_logins = [login.text for login in soup.find_all('div', class_='github_login')]
+
+### GET THE CONTRIBUTORS OF THE apache/kafka REPO ###
+n = 10
+repo = g.get_repo("apache/kafka")
+contributors = repo.get_contributors()

Review Comment:
   I'm worried about taking the top ten contributors by _lifetime_ commits, rather than by commits from the last year (as in `git shortlog --email --numbered --summary --since=2022-04-28`), since we have some prolific contributors in the past who are no longer active.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org