beautifier/js-beautify

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

Open

#741 geöffnet am 20. Juli 2015

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

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

Tech Stack
javascriptnodejs
Domain
clitooling
Issue Type
feature
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 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
JavaScriptCLI basicsNode.js
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85
Research-Richtung
Examine the main entry point of js beautify (likely in bin/js beautify.js) to understand how options are parsed and how files are processed. Look at how the current beautification is performed and where the output is written. The goal is to add a ' check' flag that runs beautification but instead of saving, compares the result to the original and fails with exit code 1 if different. Review any existing configuration options for similar behavior.