Skip to content

chore(ci): remove release-please workflow (releases run via maintainer CLI) - #539

Merged
Seth Juarez (sethjuarez) merged 1 commit into
mainfrom
sethjuarez/python-harness-correctness
Sep 10, 2026
Merged

Seth Juarez (sethjuarez) merged 1 commit into
mainfrom
sethjuarez/python-harness-correctness

Conversation

@sethjuarez

Copy link
Copy Markdown
Member

Why

The microsoft org blocks the Actions GITHUB_TOKEN from creating pull requests, and an org member cannot install a GitHub App on the org. So release-please can't run in CI here — the workflow could only ever 403 when opening the Release PR, failing on every push to main.

What changed

Removes .github/workflows/release-please.yml. The release-please config (release-please-config.json, .release-please-manifest.json) stays — it's still the source of truth.

New release flow (credential-free)

Releases are cut by running the release-please CLI under a maintainer's own identity (which the org permits — it's a real user, not the Actions token):

  1. release-please manifest-pr → opens the Release PR as the maintainer.
  2. Maintainer reviews + merges.
  3. release-please manifest-release (or gh release create python/X.Y.Z) tags the merge as the maintainer — a real-user tag push, so the existing prompty-python.yml publisher fires and publishes to PyPI via OIDC.

No GitHub App, no PAT, no org-owner approval, and no reliance on the Actions token for PR creation or tag-triggered publishing.

This is the mechanism already in use for the first Python 2.0.0 release (#538).

Co-authored-by: Copilot App 223556219+Copilot@users.noreply.github.com

…r CLI

The microsoft org blocks the Actions GITHUB_TOKEN from opening pull requests,
and an org member cannot install a GitHub App on the org, so release-please
cannot run in CI here. Releases are instead cut by running the release-please
CLI under a maintainer's own identity (which the org permits) to open the
Release PR; the existing tag-triggered OIDC publisher handles PyPI once the
maintainer tags the merge.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@sethjuarez
Seth Juarez (sethjuarez) merged commit 84120d8 into main Sep 10, 2026
14 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