[llvm] Reapply [workflows] Split pr-code-format into two parts to make it more secure (#78215) (PR #80495)

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 16 14:28:35 PST 2024


boomanaiden154 wrote:

> @jyknight @boomanaiden154 So it's OK to commit this as-is and then we switch to using the merge commit in a follow up patch?

I think it should be good. I would think any work being done to move this to checkout the default `pull_request` ref would either need to be done in this patch, or would be preferred to be done after this patch, so it should just land to make forward progress there.

This just keeps the current state of things and doesn't introduce any regressions while being more secure, other than the (somewhat) edge case where there are merge conflicts, so seems fine to land to me.

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


More information about the llvm-commits mailing list