From 4ce9577654be4b7476f42c65c0af0d38f83ba93c Mon Sep 17 00:00:00 2001 From: leftibot Date: Wed, 15 Apr 2026 16:48:41 -0600 Subject: [PATCH] Address review: run auto-clang-format on push to develop/master instead of PRs The workflow cannot push back to fork branches, so running on pull_request was fundamentally broken for fork PRs. Changed trigger to push on develop and master branches, and simplified checkout (no fork-specific options needed). Requested by @lefticus in PR #691 review. Co-Authored-By: Claude Opus 4.6 (1M context) --- .github/workflows/auto-clang-format.yml | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/.github/workflows/auto-clang-format.yml b/.github/workflows/auto-clang-format.yml index a6ea16dc..5d8a17d0 100644 --- a/.github/workflows/auto-clang-format.yml +++ b/.github/workflows/auto-clang-format.yml @@ -1,5 +1,9 @@ name: auto-clang-format -on: [pull_request] +on: + push: + branches: + - develop + - master jobs: build: @@ -7,10 +11,6 @@ jobs: steps: - uses: actions/checkout@v6 - with: - repository: ${{ github.event.pull_request.head.repo.full_name }} - ref: ${{ github.event.pull_request.head.ref }} - fetch-depth: 0 - uses: DoozyX/clang-format-lint-action@v0.20 with: source: '.'