[all-commits] [llvm/llvm-project] 99120b: [CI] Make Premerge only Comment if Tests Failed (#...

[Aiden Grossman via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 99120bb51bf728d7ba7fad5068227f8c6e707159
      https://github.com/llvm/llvm-project/commit/99120bb51bf728d7ba7fad5068227f8c6e707159
  Author: Aiden Grossman <aidengrossman at google.com>
  Date:   2025-11-21 (Fri, 21 Nov 2025)

  Changed paths:
    M .ci/premerge_advisor_explain.py

  Log Message:
  -----------
  [CI] Make Premerge only Comment if Tests Failed (#169102)

Before, we were unconditionally writing a message. After this patch, we
only write a message when the tests failed, or there is already an
existing comment. This is how this workflow was intended to work
originally, but is not how it ended up working, mostly due to my
misconceptions around how the existing code formatter pass handled this
case (we need to actually not write out any comments, not write out a
specific message).



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications