Skip to content

Internal: Grant nested build permissions in Deploy workflow [TMZ-1053] - #689

Merged
ManorHazaz merged 1 commit into
mainfrom
TMZ-1053-deploy
Aug 25, 2026
Merged

Internal: Grant nested build permissions in Deploy workflow [TMZ-1053]#689
ManorHazaz merged 1 commit into
mainfrom
TMZ-1053-deploy

Conversation

@ManorHazaz

@ManorHazaz ManorHazaz commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Grant deployments: write and pull-requests: write on the Deploy workflow's build job so it can call build.yml
  • GitHub rejects the reusable workflow at parse time when nested jobs request those scopes, even if they are skipped on workflow_dispatch

Test plan

  • Confirm deploy.yml parses (no "Invalid workflow file" on the Deploy workflow)
  • Confirm Deploy workflow_dispatch can run past the build reusable-workflow call

Made with Cursor

✨ PR Description

1. Problem & Context

Nested reusable workflows need explicit permission grants to execute restricted GitHub Actions (TMZ-1053). Without these, the build.yml job lacks authorization for deployments and PR operations.

2. What Changed (Where)

.github/workflows/deploy.yml: Added permissions block to build job granting actions: read, contents: read, deployments: write, pull-requests: write.

3. How It Works

Parent workflow (deploy.yml) delegates to reusable workflow (build.yml) via uses. Explicit permissions inherited via secrets: inherit now explicitly grant required capabilities for nested workflow execution.

4. Risks

None—minimal scope, scoped to build job only, follows GitHub best practices for nested workflows.

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Description using Guidelines Learn how

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown

Hello Elementor Build

Last updated at: 20260825.1110
Version: 3.5.0

✅ Hello Elementor build is ready for download.

You can download the latest build from the link below:

🔗 hello-elementor.3.5.0.zip

The build is available for 7 days.

@ManorHazaz
ManorHazaz merged commit f82998d into main Aug 25, 2026
45 checks passed
@ManorHazaz
ManorHazaz deleted the TMZ-1053-deploy branch August 25, 2026 11:17
ManorHazaz added a commit that referenced this pull request Aug 25, 2026
#689) (#690)

Co-authored-by: Manor Hazaz <manorhazaz@gmail.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants