microsoft / microsoft/TypeScript
Expected token errors should display all expected tokens, not just the first
Nessuno ha ancora preso questa issue.
- Lingua principale
- Go
- Stelle
- 111k
- Fork
- 14.3k
- Merge medio
- 2g 4h
- PR unite (30g)
- 132
Descrizione
🕗 Version & Regression Information
Pretty sure this has been the behaviour since the rewrite.
⏯ Playground Link
Playground link with relevant code
💻 Code
function a0([a, b, [[c]]]: [number, number, string[][]]) { }
a0([1, 2, [["world"]]);
🙁 Actual behavior
On the last ), "Expected ','."
🙂 Expected behavior
On the last ), "Expected ',' or ']'."
Right now the parser only shows the first error that starts at each location. This is an OK approach, especially when parsing has gone badly wrong, but often the parser will expect multiple tokens. The parser should report all the tokens in one error that were previously issued as multiple Expected_0 errors.
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia dalla riproduzione collegata di TypeScript Playground e segui la gestione del parser per le diagnostiche Expected_0 in corrispondenza dell'ultima parentesi tonda di chiusura. La modifica è completata quando lo stesso esempio segnala insieme la virgola attesa e la parentesi quadra di chiusura, invece di segnalare solo il primo token.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- typescript
- Ambito
- compilers
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100