Missing await causes true evaluation

Aberta
#22 0 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
4/5
Tempo estimado
3-5 dias
Facilidade para iniciantes
25/100
Tipo de issue
Bug
Clareza
Precisa de esclarecimento
Status de atividade
Estagnada
Stack de tecnologia
javascript, typescript
Domínio
api

Direção de pesquisa

Comece localizando a implementação de isEnabled e qualquer uso ou teste relacionado e, em seguida, reproduza a condicional mostrada na issue. A issue não fornece nenhum comportamento-alvo além de identificar o problema de missing-await, portanto a resolução pretendida e os critérios de conclusão precisam ser esclarecidos antes da implementação.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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.

Linguagem predominante
TypeScript
Estrelas
19
Forks
6
Merge médio
2d 3h
PRs com merge (30d)
4

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de microsoft/FeatureManagement-JavaScript

Todas as issues de microsoft/FeatureManagement-JavaScript

Issues semelhantes

Mais issues de TypeScript

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.