From cd205efb9d788f55b8051f6ed5cf01dc1bba982e Mon Sep 17 00:00:00 2001 From: Aiden Grossman Date: Wed, 18 Oct 2023 23:28:20 -0700 Subject: [PATCH] Revert "[Github] Make PR formatting job only run with C/C++ changes (#69556)" This reverts commit 80b2aac2c671771d74bc5d7426f7bd4ffa0b8a8e. I mistakenly assumed this job didn't also do python formatting (should've grepped for more than just black in the python portion of this script). Pulling it out for now to get python formatting working again while the patch is iterated further. --- .github/workflows/pr-code-format.yml | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/.github/workflows/pr-code-format.yml b/.github/workflows/pr-code-format.yml index 060646df6ae4..3a91ffb0b1ad 100644 --- a/.github/workflows/pr-code-format.yml +++ b/.github/workflows/pr-code-format.yml @@ -1,19 +1,10 @@ name: "Check code formatting" - -on: - pull_request_target: - paths: - - '**/*.cpp' - - '**/*.c' - - '**/*.h' - - '**/*.inc' - +on: pull_request_target permissions: pull-requests: write jobs: - cpp_code_formatter: - name: "Check C++ Formatting" + code_formatter: runs-on: ubuntu-latest steps: - name: Fetch LLVM sources