beautifier/js-beautify

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

Open

#741 aperta il 20 lug 2015

Vedi su GitHub
 (12 commenti) (10 reazioni) (0 assegnatari)JavaScript (1460 fork)batch import
good first issuetype: enhancement

Metriche repository

Star
 (8342 star)
Metriche merge PR
 (Merge medio 51g 1h) (16 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina la logica esistente di analisi delle opzioni CLI e salvataggio dei file in js beautify. Cerca dove l'output abbellito viene riscritto nel file. Introduci una nuova opzione (es. check) che confronta il contenuto originale e quello abbellito, esce con codice diverso da zero se diversi e non modifica il file.
Tech stack
javascript
Dominio
cli
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75