Skip to content

fix(ci): grant contents: write in main-build.yml#1092

Merged
cjpais merged 1 commit intocjpais:mainfrom
VirenMohindra:vm/fix-ci-main-build-permissions
Mar 19, 2026
Merged

fix(ci): grant contents: write in main-build.yml#1092
cjpais merged 1 commit intocjpais:mainfrom
VirenMohindra:vm/fix-ci-main-build-permissions

Conversation

@VirenMohindra
Copy link
Copy Markdown
Contributor

@VirenMohindra VirenMohindra commented Mar 19, 2026

Before Submitting This PR

  • I have searched existing issues and pull requests to ensure this isn't a duplicate
  • I have read CONTRIBUTING.md

Human Written Description

The main-build.yml I added in #1091 has a startup_failure on first run. GitHub enforces that when a calling workflow job specifies permissions, those permissions must be >= what the called reusable workflow declares — build.yml declares contents: write, so the caller must also grant contents: write or GitHub won't start the jobs at all. Missed this during local testing since actionlint doesn't catch it.

Related Issues/Discussions

Fixes startup_failure introduced by #1091

Testing

  • push to main triggers Main Branch Build without startup_failure
  • all 7 platform matrix jobs queue and run

AI Assistance

  • AI was used

  • Tools used: Claude Code

  • How extensively: identified and fixed the permissions mismatch

@cjpais cjpais merged commit c5ec92b into cjpais:main Mar 19, 2026
2 checks passed
@VirenMohindra VirenMohindra deleted the vm/fix-ci-main-build-permissions branch March 19, 2026 06:43
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