[all-commits] [llvm/llvm-project] 56cada: [Workflow] Expand code-format-helper.py error repo...

Ryan Prichard via All-commits all-commits at lists.llvm.org
Thu Oct 26 19:13:33 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 56cadac85b02f52ba3e6a84f2cdd73e3b43070ac
      https://github.com/llvm/llvm-project/commit/56cadac85b02f52ba3e6a84f2cdd73e3b43070ac
  Author: Ryan Prichard <rprichard at google.com>
  Date:   2023-10-26 (Thu, 26 Oct 2023)

  Changed paths:
    M llvm/utils/git/code-format-helper.py

  Log Message:
  -----------
  [Workflow] Expand code-format-helper.py error reporting (#69686)

* Consider the darker/clang-format command to have failed if the exit
   code is non-zero, regardless of the stdout/stderr output.
* Propagate stderr from the formatter command to the script's
   caller (and into the GitHub log).
* On success, dump stdout to the caller, so it ends up in GitHub's log.
   I'm not sure what this would ever be, but if it exists, it should be
   preserved.
* Just before the script exits, if any formatter failed, print a line
   showing which formatters failed.




More information about the All-commits mailing list