[llvm] Reland pr code format updates (PR #71131)

Tom Stellard via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 7 16:40:33 PST 2023


tstellar wrote:

> Currently it just fails in the step where it checks if the PR is mergeable. Would it be preferable to make it adjust the comment with a message?

Yeah, I think it would be useful to produce a message like: "Failed to check code formatting, please rebase your PR on the  main branch to verify the formatting."

I think most people get confused if they see a failed test, so we should try to help as much as possible.

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


More information about the llvm-commits mailing list