From efa1048d42bcb6e72056dcb42ef1a999706151a7 Mon Sep 17 00:00:00 2001 From: Alex Waygood Date: Tue, 8 Nov 2022 16:33:17 +0000 Subject: [PATCH] stubsabot: warn if stubtest is skipped in CI (for real this time) (#9133) Fixes a bug in #8681. The logic currently is broken; there should have been a warning message in the PR body for #9118, but there wasn't. --- scripts/stubsabot.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/stubsabot.py b/scripts/stubsabot.py index bb2b7bc4a..574a73611 100644 --- a/scripts/stubsabot.py +++ b/scripts/stubsabot.py @@ -578,7 +578,7 @@ def get_update_pr_body(update: Update, metadata: dict[str, Any]) -> str: if update.diff_analysis is not None: body += f"\n\n{update.diff_analysis}" - stubtest_will_run = not metadata.get("stubtest", {}).get("skip", False) + stubtest_will_run = not metadata.get("tool", {}).get("stubtest", {}).get("skip", False) if stubtest_will_run: body += textwrap.dedent( """