[refactor] use async/await syntax?
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 5.8k
- Forks
- 704
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
As I dig through the source, I'm noticing that there's a lot of messy manual Promise chaining, eg. https://github.com/nodegit/nodegit/blob/master/generate/scripts/generateNativeCode.js#L121-L171. Is there a reason not to use async/await syntax? I'd be happy to submit some PRs if maintainers would be interested.
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 par generate/scripts/generateNativeCode.js, en particulier les lignes 121–171, et examinez les chaînes de Promise environnantes. L’issue ne définit pas quel code devrait être modifié et ne fournit pas de tests ; déterminez donc d’abord si les maintainers souhaitent une conversion vers async/await et quelle portée ainsi que quels critères d’achèvement ils accepteraient.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript, nodejs
- Domaine
- tooling
- Type d'issue
- Refactorisation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 25/100