Skip to content

fix(ci): add missing permissions to deploy workflow#116

Merged
raymondk merged 1 commit intomainfrom
fix/deploy-workflow-permissions
Mar 24, 2026
Merged

fix(ci): add missing permissions to deploy workflow#116
raymondk merged 1 commit intomainfrom
fix/deploy-workflow-permissions

Conversation

@marc0olo
Copy link
Copy Markdown
Member

Summary

  • The deploy workflow has been failing with startup_failure on every push to main since feat: replace custom validator with skill-validator CLI #115 added pull-requests: write to _checks.yml
  • deploy-ic.yml calls _checks.yml as a reusable workflow but never granted the required pull-requests: write permission
  • Adds top-level and job-level permissions to match ci.yml

Last successful deploy: March 21. All 6 deploys to main since March 23 have failed.

The deploy-ic.yml workflow started failing with startup_failure after
#115 added pull-requests: write to _checks.yml. The deploy workflow
never granted this permission to the reusable workflow call, causing
GitHub to reject the workflow at startup.

Adds top-level and job-level permissions to match ci.yml.
@marc0olo marc0olo requested review from a team and JoshDFN as code owners March 24, 2026 14:05
@github-actions
Copy link
Copy Markdown

Skill Validation Report

No skill files were changed in this PR — validation skipped.

@raymondk raymondk enabled auto-merge (squash) March 24, 2026 14:07
@raymondk raymondk merged commit 1b59056 into main Mar 24, 2026
6 checks passed
@raymondk raymondk deleted the fix/deploy-workflow-permissions branch March 24, 2026 14:07
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