Skip to content

chore: Update release.yml to include permissions#119

Merged
sighphyre merged 1 commit intomainfrom
gastonfournier-patch-1
Mar 16, 2026
Merged

chore: Update release.yml to include permissions#119
sighphyre merged 1 commit intomainfrom
gastonfournier-patch-1

Conversation

@gastonfournier
Copy link
Contributor

About the changes

This id-token might be required to publish to npm

Added permissions for contents and id-token in release workflow
Copilot AI review requested due to automatic review settings March 16, 2026 10:25
@gastonfournier gastonfournier self-assigned this Mar 16, 2026
Copy link
Member

@sighphyre sighphyre left a comment

Choose a reason for hiding this comment

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

LG

@github-project-automation github-project-automation bot moved this from New to Approved PRs in Issues and PRs Mar 16, 2026
@sighphyre sighphyre merged commit 931b695 into main Mar 16, 2026
4 checks passed
@sighphyre sighphyre deleted the gastonfournier-patch-1 branch March 16, 2026 10:26
@github-project-automation github-project-automation bot moved this from Approved PRs to Done in Issues and PRs Mar 16, 2026
Copy link

Copilot AI left a comment

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 updates the GitHub Actions release workflow to explicitly grant permissions needed for npm publishing scenarios that may rely on GitHub OIDC tokens.

Changes:

  • Adds explicit workflow permissions for contents: read and id-token: write.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment on lines +7 to +9
permissions:
contents: read
id-token: write

permissions:
contents: read
id-token: write
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants