[Bug]: eslint-plugin-react-hooks documentation says that 2 rules are recommended and all others are opt-in but that's not true
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 11.8k
- Forks
- 7.9k
- Merge medio
- 1 d 11 h
- PR fusionados (30 d)
- 11
Descripción
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
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Compara la documentación de eslint-plugin-react-hooks en react.dev/reference/eslint-plugin-react-hooks con la configuración referenciada en packages/eslint-plugin-react-hooks/src/index.ts alrededor de las líneas 49-52. Determina si la configuración recomendada o la documentación es incoherente; después, actualiza la fuente relevante y verifica que el comportamiento documentado de la regla coincida con la configuración.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- eslint, javascript, react
- Área
- documentation, tooling
- Tipo de issue
- Documentación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 45/100