From a8700bcca90169ecd3c40788e173f599ac326539 Mon Sep 17 00:00:00 2001 From: Boromir Date: Fri, 8 May 2026 11:25:07 -0700 Subject: [PATCH] fix(ci): push README sync to dev instead of main MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Branch protection blocks direct pushes to main. Change the git push target from HEAD (main) to origin HEAD:dev so the README update flows through the normal dev→main release cycle. Closes #269 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- .github/workflows/blog-readme-sync.yml | 2 +- .squad/agents/boromir/history.md | 12 ++++++++++++ 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/.github/workflows/blog-readme-sync.yml b/.github/workflows/blog-readme-sync.yml index 68ca82ab..75b398b7 100644 --- a/.github/workflows/blog-readme-sync.yml +++ b/.github/workflows/blog-readme-sync.yml @@ -78,5 +78,5 @@ jobs: git diff --quiet README.md || ( git add README.md && git commit -m "docs: sync Dev Blog section from docs/blog/index.md [skip ci]" && - git push + git push origin HEAD:dev ) diff --git a/.squad/agents/boromir/history.md b/.squad/agents/boromir/history.md index 47cd46ad..8abe7659 100644 --- a/.squad/agents/boromir/history.md +++ b/.squad/agents/boromir/history.md @@ -48,6 +48,18 @@ ## Learnings +### 2026-05-XX — Issue #269: Blog → README Sync workflow branch protection fix + +**Problem:** `blog-readme-sync.yml` pushed directly to `main` after updating `README.md`, which is blocked by branch protection rules (direct pushes forbidden, "Build Solution" check required). + +**Fix (Option C):** Changed `git push` to `git push origin HEAD:dev` in the "Commit updated README" step. The workflow still triggers on `push: branches: [main]` (reading `docs/blog/index.md` from main), but the README update is pushed to `dev` — the normal development branch — and flows through the standard dev→main release cycle. + +**Key insight:** The `permissions: contents: write` block was already present. No new secrets or PAT bypass needed. One-line change. + +**Decision:** Captured in `.squad/decisions/inbox/boromir-269-readme-sync-target.md`. + +--- + ### 2026-05-08 — Issue #249: AppHost Mongo Clear Hardening **What was done:**