[CI] clang-format action broken

[nikic]

The code formatting action appears to be broken, and no longer reports incorrect formatting. See e.g. #97029, but I've noticed this in multiple PRs recently. Not sure when exactly it broke.

[llvmbot]

@llvm/issue-subscribers-infrastructure

Author: Nikita Popov (nikic)

The code formatting action appears to be broken, and no longer reports incorrect formatting. See e.g. https://github.com//pull/97029, but I've noticed this in multiple PRs recently. Not sure when exactly it broke.

[tstellar]

Maybe this one: 5914a56 ?

It's not just that the comment is missing, the job is passing even if there is incorrect formatting, right?

[nikic]

Yeah, the job is passing.

7620fe0 is another possible culprit.

[boomanaiden154]

I'll take a look.

[boomanaiden154]

It doesn't even seem like the job is running currently? There are quite a few other jobs missing in new PRs as well. The last code formatting job currently ran 25+ minutes ago and PRs have definitely been updated more recently.

I also can't get the job to run on my fork currently.

It seems like maybe just pull_request jobs? All the pull_request_target jobs, like the PR labelers, seem to be working fine.

[boomanaiden154]

https://www.githubstatus.com/incidents/9vwllhs2w1kj

I guess I'll wait for that to be fixed and then get to investigating.

[tstellar]

Here is a job from a few hours ago that correctly failed: https://github.com/llvm/llvm-project/actions/runs/9714796529

[boomanaiden154]

It correctly failed on my fork too: https://github.com/boomanaiden154/llvm-project/actions/runs/9716540595/job/26820327869?pr=48

I wonder if it's the one cpp file case?

[boomanaiden154]

It's 7620fe0.

It breaks at least the one cpp file case.

I'll revert to get things back to the way they were and ping on the PR.