From f4cd1da1c69887624ccdb1d73b651614e94d2b68 Mon Sep 17 00:00:00 2001 From: JerrettDavis Date: Wed, 23 Sep 2026 00:34:11 -0500 Subject: [PATCH] fix(ci): validate PR title only in conventional-commits check --- .github/workflows/conventional-commits.yml | 16 +++------------- 1 file changed, 3 insertions(+), 13 deletions(-) diff --git a/.github/workflows/conventional-commits.yml b/.github/workflows/conventional-commits.yml index a15d1e2d..59c6ba8a 100644 --- a/.github/workflows/conventional-commits.yml +++ b/.github/workflows/conventional-commits.yml @@ -28,19 +28,9 @@ jobs: failures.push(`PR title: ${title}`); } - const commits = await github.paginate(github.rest.pulls.listCommits, { - owner: context.repo.owner, - repo: context.repo.repo, - pull_number: context.payload.pull_request.number, - per_page: 100, - }); - - for (const commit of commits) { - const subject = commit.commit.message.split('\n')[0]; - if (!allowed.test(subject) && !allowedSpecial.test(subject)) { - failures.push(`${commit.sha.substring(0, 7)}: ${subject}`); - } - } + // Only the PR title is validated. Linting every intermediate commit too broke + // automated PRs whose fixup commits (e.g. from Copilot's CI-failure auto-fix) + // aren't conventional themselves, even though the PR title is compliant. if (failures.length > 0) { core.setFailed([