kentcdodds/kcd-scripts

lintstagedrc.js failes on Windows

Open

#70 aperta il 2 ott 2018

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (206 fork)github user discovery
bughelp wanted

Metriche repository

Star
 (888 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

  • node version: 10.10.0
  • npm (or yarn) version: 6.0.0

Relevant code or config

npm run precommit

Problem description:

I'm working on a fork of kcd-scripts for my own company's purposes and while testing the lintstagedrc.js file failes on Windows. Probably has to do with #8 or is very similar. I also noticed this when working on React-Testing-Library when I went to commit, again lintstagedrc.js fails.

Unfortunately I'm not versed enough in the fs and path operators to figure out how to change \ for /. Unless we write a dirty method to check for win32 and do a global replace on the path string, tranding slashes. But I don't know what other problems might arise from that.

Guida contributor

Direzione di ricerca
Esamina il file lintstagedrc.js per identificare dove vengono utilizzati i separatori di percorso con barra rovesciata invece della barra in avanti e garantisci la compatibilità multipiattaforma utilizzando path.join o path.normalize con conversione posix.
Tech stack
javascript
Dominio
clibuild system
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.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70