Skip to content

fix(ci): repair broken weekly-update workflow YAML#73

Merged
jdalton merged 3 commits intomainfrom
fix/weekly-update-workflow
Mar 25, 2026
Merged

fix(ci): repair broken weekly-update workflow YAML#73
jdalton merged 3 commits intomainfrom
fix/weekly-update-workflow

Conversation

@jdalton
Copy link
Copy Markdown
Collaborator

@jdalton jdalton commented Mar 25, 2026

Summary

  • Fix invalid YAML: the gh pr create --body multiline string had unindented lines that broke the YAML block scalar (broken since initial commit 2df33a8d)
  • Fix exit code capture: use PIPESTATUS[0] instead of $? after | tee to get claude's actual exit code
  • Fix git push auth: set remote URL with token since persist-credentials: false was added by zizmor hardening

Test plan

  • YAML lint passes (yaml-lint)
  • Trigger workflow via workflow_dispatch to verify end-to-end

🤖 Generated with Claude Code

The workflow was invalid YAML since creation — the `gh pr create --body`
multiline string had unindented continuation lines that broke the YAML
block scalar. Also fixes exit code capture (PIPESTATUS instead of $?)
after pipe to tee, and adds token-based auth for git push since
persist-credentials is disabled.
@socket-security
Copy link
Copy Markdown

socket-security bot commented Mar 25, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedgithub/​actions/​upload-artifact@​bbbca2ddaa5d8feaa63e36b76fdaad77386f024f99100100100100

View full report

@jdalton jdalton merged commit 04e344c into main Mar 25, 2026
13 checks passed
@jdalton jdalton deleted the fix/weekly-update-workflow branch March 25, 2026 15:45
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.

1 participant