feat(env): add vp env clean command#2003
Conversation
✅ Deploy Preview for viteplus-preview canceled.
|
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1efe4929b1
ℹ️ 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".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 30ec1fc29c
ℹ️ 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
left a comment
fengmk2
left a comment
There was a problem hiding this comment.
@liangmiQwQ Love this feature!
|
@liangmiQwQ can you fix conflicts? I'm going to merge this. |
liangmiQwQ
force-pushed
the
liang/codex/1983-env-clean
branch
from
fe24e82 to
4087838
Compare
2 participants
Close #1983
This pull request add a
vp env cleancommand that is used to clean all unused Node.js runtime and package managers downloaded by Vite+ and corepack.The clean order:
corepack cache clean$VP_HOME/js_runtime(Except the current use one and the default one)$VP_HOME/package_managerThe reason why
corepack cache cleanis done first is Node.js runtime cleaning may delete the runtime corepack needed, which slows down the cleaning process.This PR also adds a small note in
vp env listandvp env list-remotecommand to suggest users usevp env cleancommand. Personally, I think adding tips in allvp envsub commands can be noisy and conflict with #1799's philosophy.🤖 Generated with Codex