[PATCH] D159197: [clang][ci] Don't block all jobs on clang-format

2023-08-31 Thread Louis Dionne via Phabricator via cfe-commits
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rGe34ddb1b5da4: [clang][ci] Don't block all jobs on clang-format (authored by ldionne). Repository: rG LLVM Github Monore

[PATCH] D159197: [clang][ci] Don't block all jobs on clang-format

2023-08-31 Thread Louis Dionne via Phabricator via cfe-commits
ldionne added a comment. In D159197#4628232 , @aaron.ballman wrote: > Thank you for this! I'm not qualified to say the changes are correct (they > look sensible to me though), but I am really appreciative of the change. Yup this is mostly for awareness

[PATCH] D159197: [clang][ci] Don't block all jobs on clang-format

2023-08-30 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. Thank you for this! I'm not qualified to say the changes are correct (they look sensible to me though), but I am really appreciative of the change. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D159197/new/ https://re

[PATCH] D159197: [clang][ci] Don't block all jobs on clang-format

2023-08-30 Thread Louis Dionne via Phabricator via cfe-commits
ldionne created this revision. ldionne added a reviewer: aaron.ballman. Herald added a project: All. ldionne requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. It was expressed in some reviews that Clang folks didn't want to hold off other tes