fix(sync): Remove required labels from gh pr create#232
Conversation
The sync workflow was failing because consumer repos don't have the 'sync' and 'automation' labels created. Rather than requiring these labels to exist, remove the --label option entirely. Labels can be added manually after the PR is created if desired.
There was a problem hiding this comment.
Pull request overview
This PR fixes a sync workflow failure caused by attempting to add labels (sync and automation) that don't exist in consumer repositories. The change removes the --label option from the gh pr create command to allow PRs to be created without requiring these labels.
Key Changes:
- Removed the
--label "sync,automation"option from the gh pr create command in the sync workflow
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| *Automated by [Maint 68 Sync Consumer Repos](https://github.com/stranske/Workflows/actions/workflows/maint-68-sync-consumer-repos.yml)*" \ | ||
| --label "sync,automation" | ||
|
|
There was a problem hiding this comment.
The backslash line continuation character on line 734 needs to be removed. After removing the --label option, the --body argument is now the last option in the command, so it should not end with a backslash. The current code will cause a shell syntax error because the backslash indicates continuation but is followed by fi instead of more command arguments.
Automated Status SummaryHead SHA: 9a1ac22
Coverage Overview
Coverage Trend
Updated automatically; will refresh on subsequent CI/Docker completions. Keepalive checklistScopeNo scope information available Tasks
Acceptance criteria
|
🤖 Keepalive Loop StatusPR #232 | Agent: Codex | Iteration 0/5 Current State
🔍 Failure Classification| Error type | infrastructure | |
Automated Status Summary
Scope
Tasks
Acceptance criteria
Head SHA: 21342ec
Latest Runs: ❔ in progress — Gate
Required: gate: ❔ in progress