From bea59ef7763d30575125cc863c07f8e684670371 Mon Sep 17 00:00:00 2001 From: Mark Pearce Date: Wed, 15 Jul 2026 22:42:36 -0300 Subject: [PATCH] Simplify docs workflow: open PR only, don't auto-merge Drop the wait-for-checks-and-admin-merge step. The bot still opens (or updates) a docs PR after every merge to main, but a human merges it - removes the need to bypass the required-review rule for the bot's own PRs. Co-Authored-By: Claude Sonnet 5 --- .github/workflows/docs.yml | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/.github/workflows/docs.yml b/.github/workflows/docs.yml index 6b4bf0b1..b1409f06 100644 --- a/.github/workflows/docs.yml +++ b/.github/workflows/docs.yml @@ -29,7 +29,7 @@ jobs: - name: Generate docs run: npm run docs - - name: Open/update docs PR and merge once checks pass + - name: Open/update docs PR env: GH_TOKEN: ${{ secrets.DOCS_PAT }} run: | @@ -50,12 +50,7 @@ jobs: if [ -z "$PR_NUMBER" ]; then gh pr create --head "$BRANCH" --base main \ --title "Update generated docs" \ - --body "Automated docs regeneration from the latest push to main." - PR_NUMBER=$(gh pr list --head "$BRANCH" --base main --state open --json number --jq '.[0].number') + --body "Automated docs regeneration from the latest push to main. Review and merge when ready." + else + echo "PR #$PR_NUMBER already open, pushed new commit to it." fi - - echo "Waiting for checks on PR #$PR_NUMBER..." - gh pr checks "$PR_NUMBER" --watch --fail-fast - - # Bypass only the required-review rule; the validate check above already gated this. - gh pr merge "$PR_NUMBER" --admin --merge --delete-branch