Skip to content

chore(ci): Fix release workflow#2

Merged
actuarysailor merged 1 commit intomainfrom
ci/fix-release-workflow
Aug 12, 2025
Merged

chore(ci): Fix release workflow#2
actuarysailor merged 1 commit intomainfrom
ci/fix-release-workflow

Conversation

@actuarysailor
Copy link
Copy Markdown
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:02
Copy link
Copy Markdown

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 simplifies the GitHub Actions release workflow by removing redundant permissions configuration and eliminating the custom token setup. The changes align with GitHub Actions best practices for permissions management.

Key changes:

  • Removes redundant top-level permissions: contents: read block
  • Eliminates custom GITHUB_TOKEN environment variable in favor of default token
  • Updates file pattern matching to include all Dockerfile variants

Comment thread .github/workflows/release.yml
@actuarysailor actuarysailor merged commit d70be2c into main Aug 12, 2025
19 of 22 checks passed
@actuarysailor actuarysailor deleted the ci/fix-release-workflow branch August 12, 2025 15:06
@github-actions
Copy link
Copy Markdown

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