Skip to content

ci(release): Fix changelog-preview permissions.#1095

Merged
lucas-zimerman merged 9 commits intomainfrom
fix/add-statuses-permission
Feb 6, 2026
Merged

ci(release): Fix changelog-preview permissions.#1095
lucas-zimerman merged 9 commits intomainfrom
fix/add-statuses-permission

Conversation

@BYK
Copy link
Member

@BYK BYK commented Jan 23, 2026

The changelog-preview reusable workflow now requires statuses: write
permission to function correctly with GitHub App installations that
declare permissions statically.

The changelog-preview reusable workflow now requires `statuses: write`
permission to function correctly with GitHub App installations that
declare permissions statically.
@BYK BYK requested a review from lucas-zimerman as a code owner January 23, 2026 12:04
@lucas-zimerman lucas-zimerman changed the title ci(release): Fix changelog-preview permissions ci(release): Fix changelog-preview permissions. Jan 23, 2026
@lucas-zimerman
Copy link
Collaborator

Thank you for the PR!
The action seems to be failing: https://github.com/getsentry/sentry-capacitor/actions/runs/21285513375/workflow

Copy link
Collaborator

@lucas-zimerman lucas-zimerman left a comment

Choose a reason for hiding this comment

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

@lucas-zimerman
Copy link
Collaborator

we dont need the ready-to-merge label here since all we are validating is the changelog action itself.

@BYK BYK closed this Feb 6, 2026
@BYK BYK reopened this Feb 6, 2026
Copy link
Collaborator

@lucas-zimerman lucas-zimerman left a comment

Choose a reason for hiding this comment

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

Will see if after merged this issue gets resolved, otherwise I will revert it.

lucas-zimerman and others added 2 commits February 6, 2026 15:54
Co-authored-by: Claude <noreply@anthropic.com>
@lucas-zimerman lucas-zimerman merged commit 960ac32 into main Feb 6, 2026
8 checks passed
@lucas-zimerman lucas-zimerman deleted the fix/add-statuses-permission branch February 6, 2026 15:58
@lucas-zimerman
Copy link
Collaborator

Merge fixed the error, see example PR: #1111

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