shelljs/shelljs

which() should not call path.resolve()

Open

#658 geöffnet am 10. Feb. 2017

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (777 Forks)batch import
breakingfixhelp wantedtest coverage

Repository-Metriken

Stars
 (14.062 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 3T 12h) (2 gemergte PRs in 30 T)

Beschreibung

Not sure why we're calling path.resolve() on this line, but I don't think it's correct.

If we're given a correct absolute path, then we should return the absolute path. If we're given a correct relative path, we should return the same relative path. path.resolve() makes it so we're always returning an absolute path.

This is verified on Linux (both bash and zsh).

Contributor Guide

Research-Richtung
Überprüfe Zeile 91 in src/which.js, ersetze path.resolve() durch eine Funktion, die den Pfad unverändert zurückgibt, wenn er absolut oder relativ ist, und teste plattformübergreifend.
Tech Stack
javascript
Domain
cli
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Node.js
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85