Skip to content

Conversation

Perdiga
Copy link
Collaborator

@Perdiga Perdiga commented Jul 8, 2025

No description provided.

@Copilot Copilot AI review requested due to automatic review settings July 8, 2025 01:14
Copy link

@Copilot 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 enhances the release workflow by creating a dedicated release branch, opening and auto-merging a PR for the version bump, and updating archive/tag push permissions and tokens.

  • Introduces a release/v${version} branch for version bumps, creates a PR via peter-evans/create-pull-request, and auto-merges it with gh CLI
  • Adds pull-requests: write permission and fetch-depth: 0 for full history
  • Renames the PyPI secret from PYPI_API_TOKEN to PYPI_TOKEN
Comments suppressed due to low confidence (3)

.github/workflows/release.yml:103

  • The workflow no longer runs tests before merging the version bump. It’s safer to re-add a poetry run pytest step immediately after creating the release branch and before enabling auto-merge to catch any regressions.
        git checkout -b release/v${{ github.event.inputs.version }}

.github/workflows/release.yml:146

  • [nitpick] Ensure the gh CLI picks up the GH_TOKEN for authentication; you may need to set GITHUB_TOKEN or run gh auth setup-git beforehand to guarantee the CLI is authorized to merge.
        GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/release.yml:165

  • The secret name for the PyPI token was changed; please update any related documentation or README to reflect that users must now define PYPI_TOKEN instead of PYPI_API_TOKEN.
        POETRY_PYPI_TOKEN_PYPI: ${{ secrets.PYPI_TOKEN }}

@Perdiga Perdiga merged commit f1e0617 into main Jul 8, 2025
7 checks passed
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.

1 participant