stubsabot: warn if stubtest is skipped in CI (for real this time) (#9133)

AlexWaygood · web-flow · commit efa1048d42bc · 2022-11-08T08:33:17.000-08:00
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.
diff --git 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(
             """
