jsx-eslint/eslint-plugin-react

Detect uninitialized states and statefulness

Open

#1.778 aberto em 30 de abr. de 2018

Ver no GitHub
 (7 comments) (0 reactions) (0 assignees)JavaScript (2.797 forks)batch import
help wantednew rulequestion

Métricas do repositório

Stars
 (8.630 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

First things first, thank you for making this great plugin. 🙌🏻I'm hopeful that I will be able to contribute to this project. Maybe you can tell me if my two requests below make sense. Or maybe there are some rules which I do the job that I haven't discovered yet.

  • PureComponent detection works well but not the other way around: there are no errors thrown for using setState in a component declared as extends React.PureComponent.
  • In React 16.3, it eslint should warn if we have uninitialized state in a stateful component.

Guia do colaborador

Direção de pesquisa
Examine as regras existentes, como 'no direct mutation state' e 'no unused state', para entender os padrões. Implemente duas novas regras: uma para avisar quando setState é usado em um PureComponent (deve usar comparação de props), e outra para avisar quando o estado não é inicializado em um componente de classe que usa setState.
Pilha de tecnologia
javascriptreact
Domain
frontend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
JavaScriptReactESLint
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60