[Bug]: eslint-plugin-react-hooks documentation says that 2 rules are recommended and all others are opt-in but that's not true
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- JavaScript
- Estrelas
- 11.8k
- Forks
- 7.9k
- Merge médio
- 1d 11h
- PRs com merge (30d)
- 11
Descrição
Summary
https://react.dev/reference/eslint-plugin-react-hooks says:
Starting in version 6.0, these rules are available to opt-in:
...
According to that page only exhaustive-deps and rules-of-hooks are recommended.
But if you look are the source code for 7.0, you can see that extending configs.flat.recommended will more or less enable ALL the rules, including the ones related to the compiler:
Either the documentation needs to be updated or there is a bug in eslint-plugin-react-hooks where the recommended flat configs enabled too many rules. I am not in a position to decide who is wrong here :).
Page
https://react.dev/reference/eslint-plugin-react-hooks
Details
No response
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Direção de pesquisa
Compare a documentação do eslint-plugin-react-hooks em react.dev/reference/eslint-plugin-react-hooks com a configuração referenciada em packages/eslint-plugin-react-hooks/src/index.ts nas proximidades das linhas 49-52. Determine se a configuração recomendada ou a documentação está inconsistente; em seguida, atualize a fonte relevante e verifique se o comportamento documentado da regra corresponde à configuração.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- eslint, javascript, react
- Domínio
- documentation, tooling
- Tipo de issue
- Documentação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Status de atividade
- Estagnada
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 45/100