kentcdodds/kcd-scripts

lintstagedrc.js failes on Windows

Open

#70 geöffnet am 2. Okt. 2018

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (206 Forks)github user discovery
bughelp wanted

Repository-Metriken

Stars
 (888 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche die lintstagedrc.js Datei, um Stellen zu identifizieren, an denen umgekehrte Schrägstriche anstelle von normalen Schrägstrichen verwendet werden, und stelle die plattformübergreifende Kompatibilität sicher, indem du path.join oder path.normalize mit POSIX Konvertierung verwendest.
Tech Stack
javascript
Domain
clibuild system
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
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
GitNode.js
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70