palantir/blueprint

[node-build-scripts] es-lint path logic is broken

Open

#5295 aperta il 10 mag 2022

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)TypeScript (2167 fork)batch import
Domain: toolingP2Type: bughelp wanted

Metriche repository

Star
 (20.263 star)
Metriche merge PR
 (Merge medio 43g 10h) (27 PR mergiate in 30 g)

Descrizione

similar to https://github.com/palantir/blueprint/issues/4942

Environment

  • Package version(s): node-build-scripts v2.0.x

Steps to reproduce

use es-lint script outside this repo, in CircleCI where JUNIT_REPORT_PATH variable is present in the environment

Actual behavior

the utility function junitReportPath() returns a path within node_modules/reports/, which is wrong

Expected behavior

The script is usable outside this repo

Possible solution

Use process.cwd() instead of __dirname

Guida contributor

Direzione di ricerca
Sostituisci ` dirname` con `process.cwd()` nella funzione `junitReportPath()`.
Tech stack
typescriptnodejs
Dominio
build systemdeveloper experience
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Node.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80