facebookarchive/prepack

Bad command-line crashes Prepack

Offen

#2.509 geöffnet am 30.08.2018

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (520 Forks)batch import
bughelp wantedlevel 1 (super easy)priority: low

Repository-Metriken

Stars
 (14.268 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Finde die Logik zur Befehlszeilenargumentanalyse im Prepack CLI Code, wahrscheinlich in lib/prepack cli.js oder ähnlich, und füge eine Validierung für das Flag debugDiagnosticSeverity hinzu, um eine benutzerfreundliche Fehlermeldung auszugeben, anstatt eine Invariantenverletzung auszulösen.
Tech Stack
javascript
Domain
cli
Issue Type
Fehler
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.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80