From 9b5ea00fed7a533a80f1299eb044630f161c0255 Mon Sep 17 00:00:00 2001 From: Shantanu <12621235+hauntsaninja@users.noreply.github.com> Date: Tue, 4 Oct 2022 15:05:17 -0700 Subject: [PATCH] stubsabot: add some debug logs (#8850) For #8778 --- scripts/stubsabot.py | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff --git a/scripts/stubsabot.py b/scripts/stubsabot.py index d67454916..fce2eb9b6 100644 --- a/scripts/stubsabot.py +++ b/scripts/stubsabot.py @@ -511,8 +511,20 @@ async def create_or_update_pull_request(*, title: str, body: str, branch_name: s def has_non_stubsabot_commits(branch: str) -> bool: assert not branch.startswith("origin/") try: - # number of commits on origin/branch that are not on branch or are + # commits on origin/branch that are not on branch or are # patch equivalent to a commit on branch + print( + "[debugprint]", + subprocess.check_output( + ["git", "log", "--right-only", "--pretty=%an %s", "--cherry-pick", f"{branch}...origin/{branch}"] + ), + ) + print( + "[debugprint]", + subprocess.check_output( + ["git", "log", "--right-only", "--pretty=%an", "--cherry-pick", f"{branch}...origin/{branch}"] + ), + ) output = subprocess.check_output( ["git", "log", "--right-only", "--pretty=%an", "--cherry-pick", f"{branch}...origin/{branch}"], stderr=subprocess.DEVNULL, @@ -529,7 +541,7 @@ class RemoteConflict(Exception): def somewhat_safe_force_push(branch: str) -> None: if has_non_stubsabot_commits(branch): - raise RemoteConflict(f"origin/{branch} has changes not on {branch}!") + raise RemoteConflict(f"origin/{branch} has non-stubsabot changes that are not on {branch}!") subprocess.check_call(["git", "push", "origin", branch, "--force"])