Skip to content

fix: show node version fallback in vp --version#1032

Merged
fengmk2 merged 11 commits intomainfrom
fix/node-in-vp-version
Mar 19, 2026
Merged

fix: show node version fallback in vp --version#1032
fengmk2 merged 11 commits intomainfrom
fix/node-in-vp-version

Conversation

@TheAlexLichter
Copy link
Member

Resolves #1020

@netlify
Copy link

netlify bot commented Mar 18, 2026

Deploy Preview for viteplus-preview canceled.

Name Link
🔨 Latest commit b14c90d
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/69bc22820cb0aa0008dabc7b

@TheAlexLichter TheAlexLichter marked this pull request as draft March 18, 2026 17:59
@TheAlexLichter TheAlexLichter force-pushed the fix/node-in-vp-version branch 2 times, most recently from 6f47cb6 to db981c5 Compare March 18, 2026 23:26
@TheAlexLichter TheAlexLichter marked this pull request as ready for review March 18, 2026 23:42
@TheAlexLichter TheAlexLichter requested a review from fengmk2 March 19, 2026 00:21
@fengmk2
Copy link
Member

fengmk2 commented Mar 19, 2026

@codex review

@fengmk2
Copy link
Member

fengmk2 commented Mar 19, 2026

@TheAlexLichter Thanks! 😄

@fengmk2 fengmk2 self-assigned this Mar 19, 2026
Copy link

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

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: 468b94d131

ℹ️ 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".

@fengmk2
Copy link
Member

fengmk2 commented Mar 19, 2026

@TheAlexLichter should resolve conflicts then rerun the snap tests.

@TheAlexLichter TheAlexLichter force-pushed the fix/node-in-vp-version branch from d951b87 to b7010ae Compare March 19, 2026 15:20
@TheAlexLichter TheAlexLichter force-pushed the fix/node-in-vp-version branch from b7010ae to d951b87 Compare March 19, 2026 15:21
@TheAlexLichter TheAlexLichter force-pushed the fix/node-in-vp-version branch from d951b87 to dcadf1f Compare March 19, 2026 15:32
Copy link
Member

@fengmk2 fengmk2 left a comment

Choose a reason for hiding this comment

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

@TheAlexLichter I will fix the flaky snap test before I mergeit

@fengmk2 fengmk2 merged commit 2dfc0c3 into main Mar 19, 2026
33 of 35 checks passed
@fengmk2 fengmk2 deleted the fix/node-in-vp-version branch March 19, 2026 16:38
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.

vp --version not showing installed Node version

2 participants