clan/clan-infra
10
0
Fork 1
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

clan-merge: fix crash on PRs with assignees = None
All checks were successful
build / test (push) Successful in 9s
Details

Browse Source
This commit is contained in:
DavHau 2023-07-26 16:04:33 +02:00
parent 6168cb7229
commit 9127560a78
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
pkgs/clan-merge/clan_merge/__init__.py
View File

@ -31,13 +31,14 @@ def is_ci_green(pr: dict) -> bool:
def decide_merge(pr: dict, allowed_users: list[str], bot_name: str) -> bool:
assignees = pr["assignees"] if pr["assignees"] else []
if (
pr["user"]["login"] in allowed_users
and pr["mergeable"] is True
and not pr["title"].startswith("WIP:")
and pr["state"] == "open"
# check if bot is assigned
and any(reviewer["login"] == bot_name for reviewer in pr["assignees"])
and any(assignee["login"] == bot_name for assignee in assignees)
and is_ci_green(pr)
):
return True
@ -124,7 +125,6 @@ def clan_merge(
Do="merge",
)
data_encoded = json.dumps(data).encode("utf8")
print(data)
req = urllib.request.Request(
url, data=data_encoded, headers={"Content-Type": "application/json"}
)