From 1c1b3b94ac715c19f208cc35d4ee187631567caa Mon Sep 17 00:00:00 2001 From: bougyman's bot Date: Sat, 5 Sep 2026 01:07:54 -0400 Subject: [PATCH] ci: remove Validate Commit Subjects compatibility job (EXT-33) Branch protection now requires only Test, which runs the full mix precommit gate including commit-range and PR-title validation. The Validate Commit Subjects job was a temporary bridge retained while both checks were required; its purpose is now fully covered by the consolidated Test check. Co-Authored-By: Claude Sonnet 4.6 --- .github/workflows/ci.yaml | 30 ------------------------------ 1 file changed, 30 deletions(-) diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index fee9975..74a23a4 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -198,33 +198,3 @@ jobs: timeout-minutes: 2 run: ../ci/test_burrito_shared_loader.sh ./burrito_out/lc_linux_x86_64 - conventional_commits: - # Temporary compatibility job retained while branch protection still lists - # "Validate Commit Subjects" as a required check. The Test job above runs - # the same ci/validate_commit_range.sh via mix precommit and is the - # authoritative gate; this job will be removed in EXT-33 after the required- - # check set is migrated to "Test" only. - # Run for all direct triggers (pull_request, workflow_dispatch); skip only - # when main.yaml calls this workflow for a non-PR event and explicitly sets - # pull_request_title_required to false — those are post-merge runs where - # there is no open PR to gate on Validate Commit Subjects. - if: github.event_name != 'workflow_call' || inputs.pull_request_title_required - name: Validate Commit Subjects - runs-on: ubuntu-latest - steps: - - - uses: actions/checkout@v7 - with: - fetch-depth: 0 - # pull_request's default ref is the synthetic refs/pull/N/merge test-merge - # commit (subject "Merge into "), not the PR branch tip - check - # out the real head SHA instead so that commit is never in the validated - # range. Falls back to github.sha for non-PR triggers (workflow_dispatch/ - # workflow_call), where github.event.pull_request is unset. - ref: ${{ github.event.pull_request.head.sha || github.sha }} - - - env: - BASE_REF: >- - ${{ inputs.base_ref != '' && inputs.base_ref - || github.event.pull_request.base.sha }} - run: ./ci/validate_commit_range.sh