voidzero-dev / voidzero-dev/vite-plus
Do not resolve project-level Node.js version for global pm commands
- Vorherrschende Sprache
- Rust
- Sterne
- 5.8k
- Forks
- 262
- Ø Merge
- 23 Std. 41 Min.
- Gemergte PRs (30 T.)
- 138
Beschreibung
### Description
For now, `vp add -g`, `vp install -g`, `vp update -g` use the node version returned by `resolve_version`. which will care about project level's node version like `.node-version` file and even `package.json#engines.node`.
Semantically speaking, would it make more sense to always use global node version (set with `vp env default`)?
### Suggested solution
Skip project-level's node version, the resolve order is like:
1. *`VP_NODE_VERSION` env var (session override from `vp env use`) and `.session-node-version` file (session override written by `vp env use` for shell-wrapper-less environments)* **⬅️ Should we keep this layer? I prefer ignore them actually.**
2. User default from config.json
3. Latest LTS version
### Alternative
N / A
### Additional context
I'm willing to help with it, it may be partially related to #1598
### Validations
- [x] Read the [Contributing Guidelines](https://github.com/voidzero-dev/vite-plus/blob/main/CONTRIBUTING.md).
- [x] Confirm this request is for Vite+ itself and not for Vite, Vitest, tsdown, Rolldown, or Oxc.
- [x] Check that there isn't already an issue requesting the same feature.
Beitragsleitfaden
Rechercherichtung
Beginne bei den globalen Befehlswegen `vp add -g`, `vp install -g` und `vp update -g` und verfolge deren Verwendung von `resolve_version`. Prüfe, wie `.node-version`, `package.json#engines.node`, `VP_NODE_VERSION`, `.session-node-version` und `config.json` die Auflösung beeinflussen; abgeschlossen ist die Aufgabe, wenn globale Befehle der vorgesehenen Reihenfolge für globale Versionen folgen, ohne Einstellungen auf Projektebene.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- node.js
- Bereich
- cli
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Aktiv
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 58/100