Skip to content

Conversation

@actuarysailor
Copy link
Owner

Put an x into the box if that apply:

  • This PR introduces breaking change.
  • This PR fixes a bug.
  • This PR adds new functionality.
  • This PR enhances existing functionality.

Description of your changes

This PR updates the .github/workflows/release.yml workflow to:

  • Remove the redundant top-level permissions: contents: read block.
  • Ensure the workflow uses only job-level permissions (contents: write, issues: write, pull-requests: write) for the release job.
  • Remove the custom GITHUB_TOKEN environment variable, so the default GitHub Actions token is used for semantic-release.

These changes simplify permissions management and follow GitHub Actions best practices.

How can we test changes

  • Run the release workflow on a push to main or master and verify that semantic-release can create releases and push tags using the default GITHUB_TOKEN.
  • Confirm that no permission errors occur and that releases are

Copilot AI review requested due to automatic review settings August 12, 2025 15:15
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 fixes the release workflow by updating GitHub Actions permissions and token configuration. The description indicates this is meant to simplify permissions management, but the actual changes show different modifications.

  • Adds the workflow file itself to the trigger paths
  • Adds back a custom GITHUB_TOKEN environment variable with detailed comments

@actuarysailor actuarysailor merged commit 3b18d08 into main Aug 12, 2025
19 of 21 checks passed
@actuarysailor actuarysailor deleted the fix/release branch August 12, 2025 15:20
@github-actions
Copy link

This PR is included in version 1.101.0 🎉

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.

2 participants