microsoft / microsoft/FeatureManagement-JavaScript
Missing await causes true evaluation
Ouverte
Personne n'a encore pris cette issue.
- Langage dominant
- TypeScript
- Étoiles
- 19
- Forks
- 6
- Merge moyen
- 2 j 3 h
- PR mergées (30 j)
- 4
Description
if (temp.isEnabled("foo")) {
button.className = "cool";
button.textContent = "Cool Button";
} else {
button.className = "lame";
button.textContent = "Lame Button";
}
We will always run the "cool" path here, because I didn't put an await on the isEnabled call. I think this is bad.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
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 localiser l’implémentation de isEnabled ainsi que toute utilisation ou tout test associé, puis reproduisez la condition présentée dans l’issue. L’issue ne fournit aucun comportement cible au-delà de l’identification du problème de missing-await ; la résolution attendue et les critères d’achèvement doivent donc être clarifiés avant l’implémentation.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript, typescript
- Domaine
- api
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 25/100