microsoft / microsoft/TypeScript
How tsconfig exclude pattern should actually work
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Go
- Sterne
- 111k
- Forks
- 14.3k
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 117
Beschreibung
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?
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginnen Sie mit dem gemeldeten Befehl tsc --listFiles und reproduzieren Sie die aufgelistete Verzeichnisstruktur mithilfe der vier exclude-Muster. Verfolgen Sie die Verarbeitung der Exclude-Muster durch den Compiler, um zu bestimmen, warum die Muster unterschiedliche Verzeichnisse auswählen, und fügen Sie für die beobachteten Fälle eine Abdeckung hinzu oder aktualisieren Sie sie und überprüfen Sie das beabsichtigte Verhalten.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- compilers
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100