scripts: Don't capture output in prepare-release-pr.py (#8805)
We don't need to access the output in the script, and if we capture it, we don't get any output on CI, so no way to see why a release failed.
This commit is contained in:
parent
c19f63d39d
commit
e729e0d1e6
|
@ -66,22 +66,16 @@ def prepare_release_pr(
|
||||||
|
|
||||||
run(
|
run(
|
||||||
["git", "config", "user.name", "pytest bot"],
|
["git", "config", "user.name", "pytest bot"],
|
||||||
text=True,
|
|
||||||
check=True,
|
check=True,
|
||||||
capture_output=True,
|
|
||||||
)
|
)
|
||||||
run(
|
run(
|
||||||
["git", "config", "user.email", "pytestbot@gmail.com"],
|
["git", "config", "user.email", "pytestbot@gmail.com"],
|
||||||
text=True,
|
|
||||||
check=True,
|
check=True,
|
||||||
capture_output=True,
|
|
||||||
)
|
)
|
||||||
|
|
||||||
run(
|
run(
|
||||||
["git", "checkout", "-b", release_branch, f"origin/{base_branch}"],
|
["git", "checkout", "-b", release_branch, f"origin/{base_branch}"],
|
||||||
text=True,
|
|
||||||
check=True,
|
check=True,
|
||||||
capture_output=True,
|
|
||||||
)
|
)
|
||||||
|
|
||||||
print(f"Branch {Fore.CYAN}{release_branch}{Fore.RESET} created.")
|
print(f"Branch {Fore.CYAN}{release_branch}{Fore.RESET} created.")
|
||||||
|
@ -92,17 +86,13 @@ def prepare_release_pr(
|
||||||
print("Running", " ".join(cmdline))
|
print("Running", " ".join(cmdline))
|
||||||
run(
|
run(
|
||||||
cmdline,
|
cmdline,
|
||||||
text=True,
|
|
||||||
check=True,
|
check=True,
|
||||||
capture_output=True,
|
|
||||||
)
|
)
|
||||||
|
|
||||||
oauth_url = f"https://{token}:x-oauth-basic@github.com/{SLUG}.git"
|
oauth_url = f"https://{token}:x-oauth-basic@github.com/{SLUG}.git"
|
||||||
run(
|
run(
|
||||||
["git", "push", oauth_url, f"HEAD:{release_branch}", "--force"],
|
["git", "push", oauth_url, f"HEAD:{release_branch}", "--force"],
|
||||||
text=True,
|
|
||||||
check=True,
|
check=True,
|
||||||
capture_output=True,
|
|
||||||
)
|
)
|
||||||
print(f"Branch {Fore.CYAN}{release_branch}{Fore.RESET} pushed.")
|
print(f"Branch {Fore.CYAN}{release_branch}{Fore.RESET} pushed.")
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue