[GH-ISSUE #79] M001/S01/T01: Add shellcheck validation and auto-merge gate to upstream-sync.yml #2

Open
opened 2026-06-05 14:28:55 -04:00 by yindo · 0 comments
Owner

Originally created by @BillyOutlast on GitHub (May 15, 2026).
Original GitHub issue: https://github.com/Heretek-AI/ProxmoxVE/issues/79

T01: Add shellcheck validation and auto-merge gate to upstream-sync.yml

Why: The existing upstream-sync.yml always creates a PR for every sync, even clean ones. Safe syncs (no unprotected conflicts + shellcheck-passing .sh files) should auto-merge automatically. Unsafe syncs need a clear diagnostic PR explaining what blocked them.

Files

  • .github/workflows/upstream-sync.yml
  • .github/workflows/scripts/validate-sync-workflow.js

Verification

  • node .github/workflows/scripts/validate-sync-workflow.js
Originally created by @BillyOutlast on GitHub (May 15, 2026). Original GitHub issue: https://github.com/Heretek-AI/ProxmoxVE/issues/79 ## T01: Add shellcheck validation and auto-merge gate to upstream-sync.yml **Why:** The existing upstream-sync.yml always creates a PR for every sync, even clean ones. Safe syncs (no unprotected conflicts + shellcheck-passing .sh files) should auto-merge automatically. Unsafe syncs need a clear diagnostic PR explaining what blocked them. ### Files - `.github/workflows/upstream-sync.yml` - `.github/workflows/scripts/validate-sync-workflow.js` ### Verification - [ ] node .github/workflows/scripts/validate-sync-workflow.js
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Heretek-AI/ProxmoxVE#2