microsoft / microsoft/TypeScript
Computing buildInfoTime even when !isIncremental
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Go
- Sterne
- 111k
- Forks
- 14.4k
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 117
Beschreibung
@sheetalkamat @johnnyreilly I'm out of my depth, but I think there's maybe a bug introduced here.
I'm getting
TypeError: Cannot read properties of undefined (reading 'includes')in fork-ts-checker-webpack-plugin when my project is NOTincremental: true.Prior to this revision
buildInfoTimewas not computed ifbuildInfoPathwasundefinedbut after this change, even thoughbuildInfoPathis undefined, it is still attempting to computets_getModifiedTime(host, buildInfoPath)Eventually, this calls to
isArtifact(undefined)in fork-ts-checker-webpack-plugin\lib\typescript\worker\lib\system.js[function isArtifact(path) {
return ((artifacts.dirs.some((dir) => path.includes(dir)) || artifacts.files.some((file) => path === file)) && artifacts.extensions.some((extension) => path.endsWith(extension)));Perhaps this needs to be handled in fork-ts-checker-webpack-plugin? But it seems to me that perhaps
buildInfoTimecould be skipped whenbuildInfoPathis unavailable.
Originally posted by @JasonKleban in dca9182
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
Beginne mit der Revision dca9182 und dem Compiler-Pfad, der ts_getModifiedTime berechnet, wenn buildInfoPath nicht verfügbar ist. Untersuche anschließend fork-ts-checker-webpack-plugin/src/typescript/worker/lib/system.ts, insbesondere isArtifact(path). Das Issue ist behoben, wenn ein nicht inkrementelles Projekt keinen undefinierten Build-Info-Pfad mehr an diese Verarbeitung übergibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- compilers
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 42/100