microsoft / microsoft/TypeScript
How tsconfig exclude pattern should actually work
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
TypeScript Version: 2.1.1 / nightly (2.2.0-dev.201xxxxx)
Say we have a following structure to specially to test exclude pattern:
/node_modules
/module/some.ts
nested/
/node_modules
/module/some.ts
/deep-nested/
/node_modules
/module/some.ts
Will check this using tsc --listFiles
How exclude patterns work:
- "node_modules" - will exclude only
/node_modules/module/some.ts - "*/node_modules" (the same effect as /node_modules/") - will exclude only
/nested/node_modules/module/some.ts - "**/node_modules" - will exclude nothing
- "**/node_modules/*" - will exclude everything (also nested, deep-nested)
Some behaviour seem strange to me
- why in case 3 it excludes nothing (considering case 2 works)?
- by default (if
excludenot specified) case 1 will be effective, shouldn't it be 4) which excludesnode_modulesin nested folders as well?
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 comando tsc --listFiles indicado y reproduce la estructura de directorios listada utilizando los cuatro patrones de exclude. Sigue el procesamiento de los patrones de exclusión del compilador para determinar por qué los patrones seleccionan directorios diferentes; después, añade o actualiza la cobertura para los casos observados y verifica el comportamiento previsto.
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