jsx-eslint/eslint-plugin-react

react/no-unused-prop-types false positive when using memo

Open

#3403 aperta il 5 set 2022

Vedi su GitHub
 (6 commenti) (4 reazioni) (0 assegnatari)JavaScript (2797 fork)batch import
bughelp wanted

Metriche repository

Star
 (8630 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Here's an example using CRA

Expected: No warning

Actual:

  • npm run lint
eslint-example/src/App.js
  23:3  warning  'showFirst' PropType is defined but prop is never used  react/no-unused-prop-types

This started when going from version 7.30.0 to 7.30.1.

  • If I remove memo the warning goes away
  • If I wrap the ternary in the return with a fragment the warning goes away but then I'd have to silence react/jsx-no-useless-fragment so not helpful either.

Guida contributor

Direzione di ricerca
Ispeziona la regola `react/no unused prop types` nel codice sorgente del plugin. Cerca le modifiche tra v7.30.0 e v7.30.1 che potrebbero aver introdotto il falso positivo quando il componente è avvolto da `React.memo`. Controlla come la regola rileva l'uso delle prop e perché `memo` la porta a non rilevare l'espressione ternaria.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptESLintReact
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40