[llvm] [CI] Make premerge_advisor_explain write comments (PR #166605)

David Spickett via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 7 02:22:56 PST 2025


================
@@ -45,13 +83,31 @@ def main(commit_sha: str, build_log_files: list[str]):
     )
     if advisor_response.status_code == 200:
         print(advisor_response.json())
+        comments = [
+            get_comment(
+                github_token,
+                pr_number,
+                generate_test_report_lib.generate_report(
+                    generate_test_report_lib.compute_platform_title(),
+                    return_code,
+                    junit_objects,
+                    ninja_logs,
+                    failure_explanations_list=advisor_response.json(),
----------------
DavidSpickett wrote:

> It doesn't currently go to the build summary. I think I'd like to keep the build summary as exactly what happened for now.

Sounds good to me. Different audiences for different things.

https://github.com/llvm/llvm-project/pull/166605


More information about the llvm-commits mailing list