jsx-eslint/eslint-plugin-react

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

Open

#3.403 aberto em 5 de set. de 2022

Ver no GitHub
 (6 comments) (4 reactions) (0 assignees)JavaScript (8.630 stars) (2.797 forks)batch import
bughelp wanted

Description

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.

Guia do colaborador

Pilha de tecnologia
javascriptreact
Domain
frontend
Tipo Issue
bug
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.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Basic React knowledgeUnderstanding of ESLint rule structure
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
55
Direção de pesquisa
The issue is a false positive in the react/no unused prop types rule when a component is wrapped in React.memo. The rule started incorrectly warning after version 7.30.0. Investigate the rule's source code in the lib/rules/no unused prop types.js file, particularly the logic that handles wrapped components. Look for changes between versions that might have introduced this regression. Review the comments for additional test cases and consider reproducing the issue with the provided example.