beautifier/js-beautify

Add option to run js-beautify as check with fail on change

Offen

#741 geöffnet am 20.07.2015

Auf GitHub ansehenContributor Guide
 (12 Kommentare) (10 Reaktionen) (0 zugewiesene Personen)JavaScript (1.460 Forks)batch import
good first issuetype: enhancement

Repository-Metriken

Stars
 (8.342 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

In order to run js-beautify as a non-intrusive pre-commit hook in git (or any other place where you wish to check that the code is beautified, but not actually change it), it would help immensely to have an option not to actually save the beautified version of a file anywhere, but to just have js-beautify fail with a non-zero exit code and some reasonable message.

Contributor Guide

Research-Richtung
Untersuche die bestehende CLI Optionsparsing und Dateispeicherlogik in js beautify. Suche, wo die verschönere Ausgabe zurück in die Datei geschrieben wird. Führe eine neue Option (z.B. check) ein, die den ursprünglichen und den verschönerten Inhalt vergleicht, mit einem Nicht Null Code beendet, wenn sie unterschiedlich sind, und die Datei nicht ändert.
Tech Stack
javascript
Domain
cli
Issue Type
Funktion
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
JavaScriptNode.js
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75