Skip to content

fix release npm compatibility#2190

Merged
wan9chi merged 3 commits into
mainfrom
agent/fix-release-npm-version
Jul 17, 2026
Merged

fix release npm compatibility#2190
wan9chi merged 3 commits into
mainfrom
agent/fix-release-npm-version

Conversation

@wan9chi

@wan9chi wan9chi commented Jul 16, 2026

Copy link
Copy Markdown
Member

Why

The v0.2.5 release run failed before publishing because npm@latest resolved to npm 12.0.1. npm 12.0.1 requires Node 22.22.2 or newer, while the workflow used Node 22.18.0.

Changes

  • run the publishing job on Node 24.18.0, which bundles npm 11.16.0
  • remove the separate global npm installation
  • add workflow_dispatch so the corrected v0.2.5 release can be started manually

@netlify

netlify Bot commented Jul 16, 2026

Copy link
Copy Markdown

Deploy Preview for viteplus-preview canceled.

Name Link
🔨 Latest commit 895d958
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/6a59776624634c0008ced277

@wan9chi wan9chi left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Context on the manual release trigger.

Comment thread .github/workflows/release.yml
@wan9chi
wan9chi marked this pull request as ready for review July 16, 2026 23:49
@wan9chi
wan9chi requested a review from cpojer July 16, 2026 23:49

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: cab49bfc69

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/release.yml

@cpojer cpojer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comment by the clanker.

@wan9chi
wan9chi requested a review from cpojer July 17, 2026 00:39

@cpojer cpojer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright, but please remove that again after the release is published.

@wan9chi
wan9chi merged commit 93dc59a into main Jul 17, 2026
42 checks passed
@wan9chi
wan9chi deleted the agent/fix-release-npm-version branch July 17, 2026 00:43
wan9chi added a commit that referenced this pull request Jul 17, 2026
## Why

[v0.2.5](https://github.com/voidzero-dev/vite-plus/releases/tag/v0.2.5)
has been published successfully by [Release run
29545637687](https://github.com/voidzero-dev/vite-plus/actions/runs/29545637687).

PR #2190 added a temporary manual trigger to retry the unpublished
release after fixing the npm/Node compatibility failure.

## Changes

- remove the temporary `workflow_dispatch` trigger
- remove the now-redundant `main` ref guard; the remaining push trigger
already only accepts `main`
wojiushixiaobai referenced this pull request in loong64/vite-plus Jul 17, 2026
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