microsoft / microsoft/TypeScript
RegExpIndicesArray - Named group indices can be undefined
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
🔎 Search Terms
RegExpIndicesArray, Regex indices
🕗 Version & Regression Information
- This is the behaviour in every version I tried, and I reviewed the FAQ for entries about Regular Expressions
⏯ Playground Link
💻 Code
const group = /a(?<mid>b)?c/d.exec('ac')!.indices!.groups!["mid"];
const index = group[0];
The typing assumes that group indices are of type [number, number], but these can also be undefined
🙁 Actual behavior
typeof group === "undefined"
hence, the type system should have complained when attempting to access via index.
🙂 Expected behavior
'group' is possibly 'undefined'
...should be reported by the type system.
Additional information about the issue
https://github.com/microsoft/TypeScript/pull/61079 fixed the same issue for the outer RegExpIndicesArray itself, but not the indices array on the matched groups
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 el ejemplo vinculado de Playground y compara la tipificación de los índices de grupo con el cambio externo de RegExpIndicesArray en el pull request #61079. El trabajo estará terminado cuando TypeScript informe de que el índice del grupo con nombre puede ser undefined para el código mostrado, manteniendo al mismo tiempo tipificaciones válidas para los grupos que participaron en la coincidencia.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript, typescript
- Área
- compilers
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Tranquilo
- Claridad
- Bien especificado
- Aptitud para principiantes
- 68/100