facebookarchive/prepack

Bad command-line crashes Prepack

Open

#2509 aperta il 30 ago 2018

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (520 fork)batch import
bughelp wantedlevel 1 (super easy)priority: low

Metriche repository

Star
 (14.268 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

This triggers an invariant violation.

node lib/prepack-cli.js --debugDiagnosticSeverity foo

It shouldn't. The command-line argument processing should rather print a user-friendly error message in such cases and exit, like it already happens for most or all other bad arguments.

Guida contributor

Direzione di ricerca
Trova la logica di parsing degli argomenti della riga di comando nel codice CLI di Prepack, probabilmente in lib/prepack cli.js o simile, e aggiungi la validazione per il flag debugDiagnosticSeverity in modo da stampare un messaggio di errore user friendly invece di innescare una violazione di invariante.
Tech stack
javascript
Dominio
cli
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80