[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