artemist/yuzu
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

ci: Abort on failure to query Github's API

Browse source 
This raises an exception if the GET request to Github's API returns anything other than 200 OK, ensuring we always have successful merges of tagged PRs.
Also, reduces the number of queried pages from 29 to 9 to reduce the number of requests.
This commit is contained in:
Morph 2023-01-29 19:19:24 -05:00
parent 5f4647df7d
commit 8532e17243
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
.ci/scripts/merge/apply-patches-by-label.py
View file

@ -20,6 +20,7 @@ def check_individual(labels):
def do_page(page): def do_page(page):
url = 'https://api.github.com/repos/yuzu-emu/yuzu/pulls?page=%s' % page url = 'https://api.github.com/repos/yuzu-emu/yuzu/pulls?page=%s' % page
response = requests.get(url) response = requests.get(url)
response.raise_for_status()
if (response.ok): if (response.ok):
j = json.loads(response.content) j = json.loads(response.content)
if j == []: if j == []:
@ -33,7 +34,7 @@ def do_page(page):
print(subprocess.check_output(["git", "commit", "-m\"Merge %s PR %s\"" % (tagline, pn)])) print(subprocess.check_output(["git", "commit", "-m\"Merge %s PR %s\"" % (tagline, pn)]))
try: try:
for i in range(1,30): for i in range(1,10):
do_page(i) do_page(i)
except: except:
traceback.print_exc(file=sys.stdout) traceback.print_exc(file=sys.stdout)