facebookarchive/prepack

Bad command-line crashes Prepack

Open

#2 509 ouverte le 30 août 2018

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (520 forks)batch import
bughelp wantedlevel 1 (super easy)priority: low

Métriques du dépôt

Stars
 (14 268 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Trouvez la logique d'analyse des arguments de ligne de commande dans le code CLI de Prepack, probablement dans lib/prepack cli.js ou similaire, et ajoutez une validation pour le drapeau debugDiagnosticSeverity afin d'afficher un message d'erreur convivial au lieu de déclencher une violation d'invariant.
Stack technique
javascript
Domaine
cli
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80