Skip to content

ci: two-stage release with manual approval (2.x)#815

Merged
fengmk2 merged 1 commit into
2.xfrom
ci/release-flow-2.x
Jun 13, 2026
Merged

ci: two-stage release with manual approval (2.x)#815
fengmk2 merged 1 commit into
2.xfrom
ci/release-flow-2.x

Conversation

@fengmk2

@fengmk2 fengmk2 commented Jun 13, 2026

Copy link
Copy Markdown
Member

Brings the 2.x maintenance branch onto the same release flow as master (#814), so every line releases the same way.

Flow: run Prepare Release (manual, with a version) -> it opens a release/vX.Y.Z PR -> merge it -> Release checks the version against npm, pushes an approval request to DingTalk, waits on the release environment gate, then publishes and creates the GitHub Release.

2.x specifics:

  • Publishes under dist-tag latest-2 (never latest).
  • Publishes lib/ directly (no build step).
  • Replaces the previous shared node-release reusable workflow.
  • npm auth via OIDC trusted publishing (id-token: write).

Requires the repo release environment (required reviewers) and npm trusted-publisher config to allow this branch's workflow. DingTalk secrets are already set repo-wide.

Replace the previous shared node-release reusable workflow with the same prepare + approve + release flow used
on master: a manually dispatched Prepare Release opens a version-bump PR, and
merging it triggers a Release that checks the version against npm, pushes an
approval request to DingTalk, waits on the `release` environment gate, then
publishes (dist-tag `latest-2`) via OIDC and creates the GitHub Release.
@fengmk2 fengmk2 self-assigned this Jun 13, 2026
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@coderabbitai

coderabbitai Bot commented Jun 13, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4a06e150-b15a-4e86-b0cf-697cdd51f301

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/release-flow-2.x

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@fengmk2 fengmk2 merged commit 8575e80 into 2.x Jun 13, 2026
4 of 28 checks passed
@fengmk2 fengmk2 deleted the ci/release-flow-2.x branch June 13, 2026 13:38
@fengmk2 fengmk2 mentioned this pull request Jun 13, 2026
fengmk2 added a commit that referenced this pull request Jun 13, 2026
Release urllib v2.44.1.

Merging this PR updates the version on `2.x` and triggers the release
workflow, which publishes to npm (dist-tag `latest-2`) and creates the
GitHub Release after manual approval.

## What's Changed

### Security

* Do not forward credential headers (`Authorization`, `Cookie`,
`Proxy-Authorization`) on cross-origin redirect, and clear
`auth`/`digestAuth` before following. Same-origin redirects are
unchanged and the caller's headers object is never mutated (#813).

### Internal

* Two-stage release workflow with manual approval, publishing the 2.x
line to the `latest-2` npm dist-tag (#815).
* Use Node 24 in the release workflow for npm 11 OIDC trusted
publishing.

---------

Co-authored-by: fengmk2 <156269+fengmk2@users.noreply.github.com>
Co-authored-by: MK <fengmk2@gmail.com>
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