microsoft / microsoft/TypeScript

Build mode handling of emitDeclarationOnly has not been generalized to cover noEmit

Ouverte
#45,476 1 commentaire 2 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Domain: Performance Domain: tsc -b
Langage dominant
Go
Étoiles
111k
Forks
14.4k
Merge moyen
1 j 19 h
PR mergées (30 j)
117

Description

Discovered while investigating #45082 - when stack.ts is updated, the rebuild reason (as of fe7962af3d20b0cefb9cf4345923d1adc82d48c7) is Status.OutputMissing because the .js, .js.map, .d.ts, and .d.ts.map files don't exist. The immediate cause is that getAllProjectOutputs is returning too many values, but the fix probably belongs deeper, e.g. in getOutputJSFileName and getEmitDeclarations (which might have unforeseen consequences).

I am not presently aware of a resulting correctness problem.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez dans src/compiler/emitter.ts avec getAllProjectOutputs et getOutputJSFileName, puis examinez getEmitDeclarations dans src/compiler/utilities.ts. Suivez l’impact de noEmit et emitDeclarationOnly sur les sorties attendues, et vérifiez que la mise à jour de stack.ts ne signale plus de sorties manquantes lorsque les fichiers correspondants ne sont intentionnellement pas émis.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
typescript
Domaine
build-system, compilers
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
42/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.