Missing await causes true evaluation

Offen
#22 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
25/100
Issue-Typ
Bug
Klarheit
Muss geklärt werden
Aktivitätsstatus
Veraltet
Tech-Stack
javascript, typescript
Bereich
api

Rechercherichtung

Beginne damit, die Implementierung von isEnabled sowie jede zugehörige Verwendung oder Tests zu finden, und bilde anschließend die im Issue gezeigte Bedingung nach. Das Issue gibt über die Identifizierung des missing-await-Problems hinaus kein Zielverhalten vor, daher müssen die beabsichtigte Lösung und die Kriterien für den Abschluss vor der Implementierung geklärt werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

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.

Vorherrschende Sprache
TypeScript
Sterne
19
Forks
6
Ø Merge
2 T. 3 Std.
Gemergte PRs (30 T.)
4

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus microsoft/FeatureManagement-JavaScript

Alle Issues in microsoft/FeatureManagement-JavaScript

Ähnliche Issues

Weitere Issues zu TypeScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.