Skip to content

Fix check-deploy-branch failure by removing fragile heredoc Python invocation#16

Draft
shielded-nate with Copilot wants to merge 4 commits into
mainfrom
copilot/enable-render-action
Draft

Fix check-deploy-branch failure by removing fragile heredoc Python invocation#16
shielded-nate with Copilot wants to merge 4 commits into
mainfrom
copilot/enable-render-action

Conversation

Copilot AI commented Jun 17, 2026

Copy link
Copy Markdown

check-deploy-branch was failing in GitHub Actions because the inline Python heredoc terminator was not parsed correctly in the YAML-indented shell block, causing an EOF syntax error. This prevented branch policy evaluation from completing and blocked deploy gating.

  • Root cause

    • Reproduced failure mode from logs: Bash could not find the PY heredoc terminator in the run: | script context.
  • Change made

    • Replaced the heredoc form (python - <<'PY' ... PY) with a single-argument inline script (python -c '...') in .github/workflows/publish.yaml.
    • Preserved branch-policy matching behavior (fnmatch against branch_policies[].name) and existing should_deploy output paths.
  • Why this resolves it

    • Eliminates heredoc delimiter parsing sensitivity in YAML-indented shell blocks while keeping the same policy evaluation logic.
# before
if POLICY_RESPONSE="${policy_response}" BRANCH_NAME="${GITHUB_REF_NAME}" python - <<'PY'
  ...
PY

# after
if POLICY_RESPONSE="${policy_response}" BRANCH_NAME="${GITHUB_REF_NAME}" python -c '
...
'

Copilot AI changed the title Enable manual Pages deploy trigger and dynamic github-pages branch policy checks Enable manual Pages deploys with dynamic github-pages branch policy gating Jun 17, 2026
Copilot AI requested a review from shielded-nate June 17, 2026 17:18
Copilot AI changed the title Enable manual Pages deploys with dynamic github-pages branch policy gating Fix check-deploy-branch failure by removing fragile heredoc Python invocation Jun 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants