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 (2.797 forks)batch import
bughelp wanted

Métricas do repositório

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

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

Direção de pesquisa
Inspecione a regra `react/no unused prop types` no código fonte do plugin. Procure alterações entre v7.30.0 e v7.30.1 que possam ter introduzido o falso positivo quando o componente é envolvido por `React.memo`. Verifique como a regra detecta o uso de props e por que o `memo` faz com que ela perca a expressão ternária.
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.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
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.
Claro
Prerequisites
JavaScriptESLintReact
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40