microsoft / microsoft/TypeScript
Duplicated files do not check unused @ts-expect-error properly
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Go
- Estrellas
- 111k
- Forks
- 14.4k
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 117
Descripción
Bug Report
🔎 Search Terms
redirectInfo, ts-expect-error
🕗 Version & Regression Information
Tested on 4.3.2, 4.9.5, and next (5.0.0-dev.20230204).
⏯ Playground Link
Reproduction: https://github.com/lamnhh/ts-expect-error-issue
💻 Code
There're 3 packages:
- package-a:
// @ts-expect-error global
export const SOMETHING: number = global.testNumber;
Anything that has an @ts-expect-error directive will work.
- a
package-bthat importspackage-a - anothe
package-cthat imports bothpackage-aandpackage-b.
Install everything, then npm link package-b to use in package-c.
🙁 Actual behavior
Unused '@ts-expect-error' directive.
> typecheck
> tsc --noEmit
../package-b/node_modules/@lamnhh/package-a/index.ts:1:1 - error TS2578: Unused '@ts-expect-error' directive.
1 // @ts-expect-error global
~~~~~~~~~~~~~~~~~~~~~~~~~~
Found 1 error in ../package-b/node_modules/@lamnhh/package-a/index.ts:1
🙂 Expected behavior
tsc in package-c passes with no error.
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 repositorio de reproducción en https://github.com/lamnhh/ts-expect-error-issue. Instala los tres paquetes, enlaza package-b con package-c y ejecuta tsc --noEmit en package-c para reproducir el diagnóstico de @ts-expect-error no utilizado de package-a. Se considera terminado cuando package-c supera la comprobación de tipos sin ese diagnóstico, mientras la directiva sigue siendo válida en package-a.
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
- 25/100