microsoft / microsoft/TypeScript
Cache `getExePath()` for performance reasons
Personne n'a encore pris cette issue.
- 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
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- 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