2016-04-25 4 views
0

So verwende ich Github Pull-Anforderungen für meine Code-Überprüfung braucht und mein einziges Problem ist, dass ich nicht sagen kann, ob eine Person Tabs oder Leerzeichen für Einzug verwendet. Wir haben hier einen Standard, und Sie können die Code-Überprüfung für die Verwendung des falschen fehlschlagen. Gibt es eine Möglichkeit zu sagen, was sie mit github verwenden oder muss ich die Datei manuell in meinem Editor öffnen, um den Unterschied zu erkennen?Abstand Zeichen während Pull Code Request Code überprüfen

+0

Verwenden Sie Travis und überprüfen Sie dort den korrekten Leerraum. Dies kann automatisch und nicht manuell überprüft werden, also warum sollte es von Hand gemacht werden? –

+0

Für diese Art von Styleguide-Adhäsion (jslint, rubocop, etc.) haben wir normalerweise sprachenspezifische Linters verwendet, und diese können in einem CI-Tool wie Travis – jake

+0

ausgeführt werden. Aufgrund von Umständen außerhalb meiner Kontrolle in meinem Unternehmen ist dies einfach keine Option für mindestens die nächsten 3 oder 4 Monate. Deshalb frage ich, ob du das mit Github machen kannst. Entschuldigung für die Verwirrung. –

Antwort

1

Gibt es eine Möglichkeit zu sagen, was sie mit Github verwenden oder muss ich die Datei in meinem Editor manuell öffnen, um den Unterschied zu erkennen?

Idealerweise - weder!

Wenn Dinge auf automatisierte Weise überprüft werden können, lassen Sie den Computer die Arbeit für Sie erledigen. Die Überprüfung der richtigen Verwendung von Leerzeichen unter vielen anderen statischen Regeln kann mit einer Vielzahl von Werkzeugen überprüft werden, die oft Linters genannt werden. Dies hängt stark davon ab, welche Sprache Ihr Projekt verwendet. Natürlich können Sie auch Ihre eigenen Scripts schreiben, wenn Sie dies wünschen.

Mit Github können Sie Ihr Repository mit einem CI-Tool wie Travis verbinden. Auf diese Weise können Sie automatisch alle Pull-Requests erstellen und Dinge wie Whitespace-Regeln überprüfen. Außerdem können Sie Testsuites, Codeformatierungen, ... ausführen - alles, was Sie automatisieren können, können Sie von dort aus ausführen (und sollten!), Um die manuelle Arbeit zu minimieren.

+0

Ich stimme dieser Antwort vollständig zu. In ungefähr einem weiteren Jahr wird dies definitiv der Fall sein. Aufgrund von Umständen, die außerhalb meiner Kontrolle bei meinem Unternehmen liegen, ist dies für mindestens die nächsten 3 oder 4 Monate keine Option. Deshalb frage ich, ob du das mit Github machen kannst. Entschuldigung für die Verwirrung. –

+0

Wenn Travis momentan keine Option ist, können Sie immer Ihre eigene Github-Integration schreiben, um ein Skript auszuführen, das diese Dinge überprüft. Aber so weit wie eigentlich der Redakteur auf Github selbst wechselt ... AFAIK, das ist nicht möglich. Unter [hier] (https://developer.github.com/v3/repos/hooks/) finden Sie die erforderlichen APIs. –

+0

Wenn es absolut auf der Github-Seite selbst sein muss und die manuelle Überprüfung in Ordnung ist, könnte es möglich sein, ein einfaches Browser-Skript/Plugin oder Javascript-Plugin zu schreiben, das falsche Leerzeichen hervorhebt. –