the-draupnir-project/Draupnir

Relative dataPath directory support implementation is relative to install directory, not current working directory

Aperta

#241 aperta il 7 feb 2024

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)TypeScript (28 fork)auto 404
L1 Very FewP1 NuisanceT3 Workflow Issuegood first issue

Metriche repository

Star
 (239 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

https://github.com/the-draupnir-project/Draupnir/blob/faee9367af5a3041368152d451e38c8d7be630dc/src/index.ts#L69

If a relative path is passed, it ends up evaluating to $DIST/../$DATA_PATH, rather than ./$DATA_PATH. This becomes an issue once you are not in the workspace (eg. you've installed Draupnir on nix, where it lives in /nix/store/awewaef3-draupnir-1.86.1/...)

Guida contributor

Direzione di ricerca
Ispeziona la riga 69 di src/index.ts per vedere come viene calcolato dataPath. Nota che l'uso di un percorso relativo alla directory del modulo ( dirname) causa il problema. La correzione dovrebbe risolvere i percorsi relativi dalla directory di lavoro corrente (process.cwd()) invece. Aggiungi un test per verificare che l'esecuzione da una directory diversa produca il percorso atteso.
Tech stack
typescript
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitNode.jsTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75