kentcdodds/kcd-scripts

lintstagedrc.js failes on Windows

Open

#70 ouverte le 2 oct. 2018

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)JavaScript (206 forks)github user discovery
bughelp wanted

Métriques du dépôt

Stars
 (888 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

  • 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.

Guide contributeur

Direction de recherche
Examinez le fichier lintstagedrc.js pour identifier les endroits où des barres obliques inverses sont utilisées au lieu de barres obliques normales et assurez la compatibilité multiplateforme en utilisant path.join ou path.normalize avec conversion posix.
Stack technique
javascript
Domaine
clibuild system
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitNode.js
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70