Microsoft/TypeScript

Improve error when parameter is missing a name

Open

#22.203 geöffnet am 27. Feb. 2018

Auf GitHub ansehen
 (6 Kommentare) (1 Reaktion) (0 zugewiesene Personen)TypeScript (6.726 Forks)batch import
Domain: Error MessagesHelp WantedSuggestion

Repository-Metriken

Stars
 (48.455 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 6T 17h) (9 gemergte PRs in 30 T)

Beschreibung

TypeScript Version: 2.8.0-dev.20180227 Code

declare class C {
    f({ x?: number }): void;
    g(): void;
}

Expected behavior:

At the ?, property in destructuring may not be marked optional.

Actual behavior:

src/a.ts(2,10): error TS1005: ',' expected.
src/a.ts(2,20): error TS1005: ',' expected.
src/a.ts(2,21): error TS1128: Declaration or statement expected.
src/a.ts(2,22): error TS1128: Declaration or statement expected.
src/a.ts(2,28): error TS1109: Expression expected.
src/a.ts(3,8): error TS1005: ';' expected.
src/a.ts(3,14): error TS1109: Expression expected.
src/a.ts(4,1): error TS1128: Declaration or statement expected.

Contributor Guide

Research-Richtung
Untersuchen Sie die Behandlung von Destrukturierungsparametern im TypeScript Parser und modifizieren Sie ihn, um vor dem Komma auf fehlende Eigenschaftsnamen zu prüfen.
Tech Stack
typescript
Domain
compilers
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TypeScript SDKparser knowledge
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30