fix: swap to just checking return code

Author: 56kyle

scripts/update-demo.py

@@ -148,11 +148,8 @@ def _create_demo_pr(demo_path: Path, branch: str, commit_start: str) -> None:
     """Creates a PR to merge the given branch into develop."""
     gh("repo", "set-default", f"{DEMO.app_author}/{DEMO.app_name}")
     search_results: subprocess.CompletedProcess = gh("pr", "list", "--state", "open", "--search", branch)
-    typer.secho(f"_create_demo_pr - {search_results.stdout=}")
 
-    if "no pull requests" not in str(search_results.stdout):
-        typer.secho(f"{str(search_results.stdout)=}")
-        typer.secho(f"no_pull_requests")
+    if search_results.returncode != 0:
         url: str = _get_pr_url(branch=branch)
         typer.secho(f"Skipping PR creation due to existing PR found for branch {branch} at {url}")
         return
@@ -172,7 +169,6 @@ def _create_demo_pr(demo_path: Path, branch: str, commit_start: str) -> None:
 def _get_pr_url(branch: str) -> str:
     """Returns the url of the current branch's PR."""
     result: subprocess.CompletedProcess = gh("pr", "view", branch, "--json", "url", "--jq", ".url")
-    typer.secho(f"_get_pr_url - {result.stdout=}")
     if result.returncode != 0:
         raise ValueError(f"Failed to find a PR URL for branch {branch}.")
     return result.stdout.strip()