Skip to content

ci: write the pull request body in the gate step - #904

Merged
svanlabs[bot] merged 2 commits into
mainfrom
fix/ci-pr-body-gate
Oct 7, 2026
Merged

svanlabs[bot] merged 2 commits into
mainfrom
fix/ci-pr-body-gate

Conversation

@svanlabs

@svanlabs svanlabs Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

The gate job wrote the PR body file in a separate step and handed its path over via GITHUB_ENV. On #903 the file was missing at scripts/check.sh full on two runs although that step reported success. The write and the check now run in the same step. Workflow-only change; the local gate does not run it.

Generated-by: claude-code/claude-haiku-5-5

🤖 Generated with Claude Code

svanlabs Bot and others added 2 commits October 7, 2026 20:54
The gate job wrote the pull request body to a file in a separate step
and passed its path on through GITHUB_ENV. On PR #903 the file was
missing when scripts/check.sh ran, on two runs in a row, although the
step that wrote it reported success. A failed gate on a correct PR is
the outcome this is meant to prevent, so the write and the check now
happen in the same step.

Generated-by: claude-code/claude-haiku-5-5

Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
The gate's roster check failed on this branch: claude-code/claude-haiku-5-5
has commits in HEAD and was not on record. Regenerated with
scripts/provenance.py report.

Generated-by: claude-code/claude-haiku-5-5

Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
@svanlabs
svanlabs Bot merged commit 2426de2 into main Oct 7, 2026
3 checks passed
@svanlabs
svanlabs Bot deleted the fix/ci-pr-body-gate branch October 7, 2026 21:03
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.

0 participants