microsoft / microsoft/TypeScript
Expected token errors should display all expected tokens, not just the first
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Go
- Estrellas
- 111k
- Forks
- 14.3k
- Merge medio
- 2 d 4 h
- PR fusionados (30 d)
- 132
Descripción
🕗 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.
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza con la reproducción vinculada de TypeScript Playground y sigue el procesamiento del parser de los diagnósticos Expected_0 en el paréntesis de cierre final. El cambio estará terminado cuando el mismo ejemplo informe de forma conjunta la coma esperada y el corchete de cierre, en lugar de solo el primer token.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- typescript
- Área
- compilers
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100