Skip to content

fix(cli): show binary location when shell config is not writable#1060

Merged
fengmk2 merged 2 commits intomainfrom
improve-shell-config-no-perms-tips
Mar 20, 2026
Merged

fix(cli): show binary location when shell config is not writable#1060
fengmk2 merged 2 commits intomainfrom
improve-shell-config-no-perms-tips

Conversation

@fengmk2
Copy link
Copy Markdown
Member

@fengmk2 fengmk2 commented Mar 20, 2026

When shell config files are read-only (e.g., managed by Nix), the
installer now shows where vp was installed and how to run it directly,
instead of a confusing permission error. On Windows, PATH failure is
no longer fatal — the installer completes and shows manual instructions.

Adds CI test for readonly shell config scenario on Linux.

When shell config files are read-only (e.g., managed by Nix), the
installer now shows where vp was installed and how to run it directly,
instead of a confusing permission error. On Windows, PATH failure is
no longer fatal — the installer completes and shows manual instructions.

Adds CI test for readonly shell config scenario on Linux.
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 20, 2026

Deploy Preview for viteplus-preview canceled.

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

Copy link
Copy Markdown
Member Author

fengmk2 commented Mar 20, 2026


How to use the Graphite Merge Queue

Add the label auto-merge to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

permissions:
contents: read
steps:
- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1

Check notice

Code scanning / zizmor

credential persistence through GitHub Actions artifacts Note test

credential persistence through GitHub Actions artifacts
@fengmk2 fengmk2 self-assigned this Mar 20, 2026
@fengmk2 fengmk2 marked this pull request as ready for review March 20, 2026 02:09
Copy link
Copy Markdown
Member Author

fengmk2 commented Mar 20, 2026

Setting up VITE+...
warn: Cannot write to /home/runner/.bash_profile (permission denied), skipping.
warn: Cannot write to /home/runner/.bashrc (permission denied), skipping.
warn: Cannot write to /home/runner/.profile (permission denied), skipping.

✔ VITE+ successfully installed!

  The Unified Toolchain for the Web.

  Get started:
    vp create       Create a new project
    vp env          Manage Node.js versions
    vp install      Install dependencies
    vp migrate      Migrate to Vite+

  Vite+ is now managing Node.js via vp env.
  Run vp env doctor to verify your setup, or vp env off to opt out.

  Run vp help to see available commands.

  note: Could not automatically add vp to your PATH.

  vp was installed to: /home/runner/.vite-plus/bin

  To use vp, add this line to your shell config file:

    . "$HOME/.vite-plus/env"

  Common config files:
    - Bash: ~/.bashrc or ~/.bash_profile
    - Zsh:  ~/.zshrc
    - Fish: source "$HOME/.vite-plus/env.fish" in ~/.config/fish/config.fish

  Or run vp directly:

    /home/runner/.vite-plus/bin/vp

@paperclover
Copy link
Copy Markdown

beautiful

@fengmk2 fengmk2 merged commit 611729c into main Mar 20, 2026
33 checks passed
@fengmk2 fengmk2 deleted the improve-shell-config-no-perms-tips branch March 20, 2026 06:00
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.

3 participants