microsoft / microsoft/TypeScript

Cache `getExePath()` for performance reasons

Ouverte
#63,801 7 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Possible Improvement
Langage dominant
Go
Étoiles
111k
Forks
14.3k
Merge moyen
2 j 4 h
PR mergées (30 j)
132

Description

Would it be possible to have `getExePath()` cache its result so we don't have to do all this lookup every time `tsgo` is invoked? The platform and arch rarely changes once installed on disk.

In one of the repositories I'm maintaining, we've inlined what `tsgo.js` does and added caching on top of it. We're not a very big monorepo, but this shaved off several seconds for us.

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 tsgo.js en localisant getExePath() et en déterminant combien de fois il est exécuté lorsque tsgo est invoqué. Vérifiez le comportement de recherche existant selon les plateformes et les architectures, puis confirmez que la mise en cache conserve l’exécutable sélectionné tout en évitant les recherches répétées. C’est terminé lorsque les invocations répétées n’effectuent plus de nouvelle recherche et que le comportement existant reste inchangé.

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

Évaluation

Stack technique
javascript
Domaine
cli, performance, tooling
Type d'issue
Refactorisation
Difficulté
3/5
Temps estimé
1-2 jours
Activité
Calme
Clarté
Plutôt claire
Accessibilité débutants
55/100

Recevez les nouvelles issues par e-mail

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