Skip to content

Update publish.yaml to include write permissions#1410

Merged
chris-rock merged 1 commit intomainfrom
mik/fix-child-workflow-permissions
Mar 6, 2026
Merged

Update publish.yaml to include write permissions#1410
chris-rock merged 1 commit intomainfrom
mik/fix-child-workflow-permissions

Conversation

@slntopp
Copy link
Copy Markdown
Member

@slntopp slntopp commented Mar 6, 2026

Add permissions for contents write in publish workflow

Add permissions for contents write in publish workflow
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 6, 2026

Test Results

  5 files  ±0   42 suites  ±0   1h 0m 57s ⏱️ + 1m 4s
393 tests ±0  393 ✅ ±0  0 💤 ±0  0 ❌ ±0 
433 runs  ±0  431 ✅ ±0  2 💤 ±0  0 ❌ ±0 

Results for commit 0993f1e. ± Comparison against base commit b93a4e4.

@chris-rock
Copy link
Copy Markdown
Member

/review

Copy link
Copy Markdown

@mondoo-code-review mondoo-code-review Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adds write permissions for contents to the release-manifests reusable workflow call, which is needed for creating releases or pushing tags.

@chris-rock chris-rock merged commit b195dcb into main Mar 6, 2026
25 of 28 checks passed
@chris-rock chris-rock deleted the mik/fix-child-workflow-permissions branch March 6, 2026 19:39
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants