From 85a5e46b9269f37172e7a04ad26a3ffcbd32dd69 Mon Sep 17 00:00:00 2001 From: Bruno Alla Date: Tue, 11 Aug 2020 16:51:34 +0100 Subject: [PATCH] Find recent authors based on merged pull requests --- scripts/update_contributors.py | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/scripts/update_contributors.py b/scripts/update_contributors.py index 206d7aab..c7337764 100644 --- a/scripts/update_contributors.py +++ b/scripts/update_contributors.py @@ -1,5 +1,6 @@ import json from pathlib import Path +from urllib.parse import urlencode import requests from jinja2 import Template @@ -50,10 +51,13 @@ class GitHub: return response.json() def iter_recent_authors(self): - commits = self.request("/repos/pydanny/cookiecutter-django/commits") - for commit in commits: - login = commit["author"]["login"] - if login not in BOT_LOGINS: + query_params = urlencode( + {"state": "closed", "sort": "updated", "direction": "desc"} + ) + pulls = self.request(f"/repos/pydanny/cookiecutter-django/pulls?{query_params}") + for pull_request in pulls: + login = pull_request["user"]["login"] + if pull_request["merged_at"] and login not in BOT_LOGINS: yield login def fetch_user_info(self, username):