Skip to content

fix(release-pipeline): give the ci job contents: read so publish stops failing - #28

Merged
GoogilyBoogily merged 1 commit into
mainfrom
DTBTWEB-1765-fix-release-pipeline-ci-permissions
Aug 18, 2026
Merged

fix(release-pipeline): give the ci job contents: read so publish stops failing#28
GoogilyBoogily merged 1 commit into
mainfrom
DTBTWEB-1765-fix-release-pipeline-ci-permissions

Conversation

@GoogilyBoogily

@GoogilyBoogily GoogilyBoogily commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

What

The ci job in release-pipeline.yml had no permissions block of its own, so it inherited the workflow's top-level permissions: {} default, which is contents: none. This broke the downstream actions

This gives the ci job its own contents: read, the same way bump-version, publish-npm, and publish-release already declare their own permissions.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes the release pipeline’s reusable-workflow invocation by granting the ci job the minimum required GITHUB_TOKEN permission (contents: read) so the called reusable workflow (.github/workflows/ci.yml) can start successfully.

Changes:

  • Add a job-level permissions block to jobs.ci granting contents: read to satisfy the nested reusable workflow’s requested permissions.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@codentacos codentacos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍 🧙

@GoogilyBoogily
GoogilyBoogily merged commit 45ee569 into main Aug 18, 2026
2 checks passed
@GoogilyBoogily
GoogilyBoogily deleted the DTBTWEB-1765-fix-release-pipeline-ci-permissions branch August 18, 2026 20:28
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.

4 participants