Merge pull request #2120 from tannewt/fix_website_updater_for_blinka
Fix website updater for other users.
This commit is contained in:
commit
28e17feb93
@ -117,7 +117,7 @@ def get_current_info():
|
|||||||
current_info[info["id"]] = info
|
current_info[info["id"]] = info
|
||||||
return git_info, current_info
|
return git_info, current_info
|
||||||
|
|
||||||
def create_pr(changes, updated, git_info):
|
def create_pr(changes, updated, git_info, user):
|
||||||
commit_sha, original_blob_sha = git_info
|
commit_sha, original_blob_sha = git_info
|
||||||
branch_name = "new_release_" + changes["new_release"]
|
branch_name = "new_release_" + changes["new_release"]
|
||||||
|
|
||||||
@ -138,7 +138,7 @@ def create_pr(changes, updated, git_info):
|
|||||||
languages = ""
|
languages = ""
|
||||||
if changes["new_languages"]:
|
if changes["new_languages"]:
|
||||||
languages = "New languages:\n* " + "\n* ".join(changes["new_languages"])
|
languages = "New languages:\n* " + "\n* ".join(changes["new_languages"])
|
||||||
message = "Automated website update for release {} by AdaBot.\n\n{}\n\n{}\n".format(
|
message = "Automated website update for release {} by Blinka.\n\n{}\n\n{}\n".format(
|
||||||
changes["new_release"],
|
changes["new_release"],
|
||||||
boards,
|
boards,
|
||||||
languages
|
languages
|
||||||
@ -148,7 +148,7 @@ def create_pr(changes, updated, git_info):
|
|||||||
"ref": "refs/heads/" + branch_name,
|
"ref": "refs/heads/" + branch_name,
|
||||||
"sha": commit_sha
|
"sha": commit_sha
|
||||||
}
|
}
|
||||||
response = github.post("/repos/adafruit-adabot/circuitpython-org/git/refs", json=create_branch)
|
response = github.post("/repos/{}/circuitpython-org/git/refs".format(user), json=create_branch)
|
||||||
if not response.ok and response.json()["message"] != "Reference already exists":
|
if not response.ok and response.json()["message"] != "Reference already exists":
|
||||||
print("unable to create branch")
|
print("unable to create branch")
|
||||||
print(response.text)
|
print(response.text)
|
||||||
@ -161,14 +161,14 @@ def create_pr(changes, updated, git_info):
|
|||||||
"branch": branch_name
|
"branch": branch_name
|
||||||
}
|
}
|
||||||
|
|
||||||
response = github.put("/repos/adafruit-adabot/circuitpython-org/contents/_data/files.json", json=update_file)
|
response = github.put("/repos/{}/circuitpython-org/contents/_data/files.json".format(user), json=update_file)
|
||||||
if not response.ok:
|
if not response.ok:
|
||||||
print("unable to post new file")
|
print("unable to post new file")
|
||||||
print(response.text)
|
print(response.text)
|
||||||
return
|
return
|
||||||
pr_info = {
|
pr_info = {
|
||||||
"title": pr_title,
|
"title": pr_title,
|
||||||
"head": "adafruit-adabot:" + branch_name,
|
"head": user + ":" + branch_name,
|
||||||
"base": "master",
|
"base": "master",
|
||||||
"body": message,
|
"body": message,
|
||||||
"maintainer_can_modify": True
|
"maintainer_can_modify": True
|
||||||
@ -198,9 +198,12 @@ def update_downloads(boards, release):
|
|||||||
def print_active_user():
|
def print_active_user():
|
||||||
response = github.get("/user")
|
response = github.get("/user")
|
||||||
if response.ok:
|
if response.ok:
|
||||||
print("Logged in as {}".format(response.json()["login"]))
|
user = response.json()["login"]
|
||||||
|
print("Logged in as {}".format(user))
|
||||||
|
return user
|
||||||
else:
|
else:
|
||||||
print("Not logged in")
|
print("Not logged in")
|
||||||
|
return None
|
||||||
|
|
||||||
def generate_download_info():
|
def generate_download_info():
|
||||||
boards = {}
|
boards = {}
|
||||||
@ -214,7 +217,7 @@ def generate_download_info():
|
|||||||
"new_languages": []
|
"new_languages": []
|
||||||
}
|
}
|
||||||
|
|
||||||
print_active_user()
|
user = print_active_user()
|
||||||
|
|
||||||
sha, this_version = get_version_info()
|
sha, this_version = get_version_info()
|
||||||
|
|
||||||
@ -271,8 +274,8 @@ def generate_download_info():
|
|||||||
|
|
||||||
changes["new_languages"] = set(languages) - previous_languages
|
changes["new_languages"] = set(languages) - previous_languages
|
||||||
|
|
||||||
if changes["new_release"]:
|
if changes["new_release"] and user:
|
||||||
create_pr(changes, current_info, git_info)
|
create_pr(changes, current_info, git_info, user)
|
||||||
else:
|
else:
|
||||||
print("No new release to update")
|
print("No new release to update")
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user