jsx-eslint/eslint-plugin-react

Ignore nested components in no-multi-comp

Open

#2842 aperta il 24 ott 2020

Vedi su GitHub
 (2 commenti) (1 reazione) (0 assegnatari)JavaScript (2797 fork)batch import
help wanted

Metriche repository

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

Descrizione

I don’t think react/no-multi-comp should trigger on nested components or should have an option to disable it. A practical example:

import { FormattedMessage } from 'react-intl';

export function App() {
  return (
    <FormattedMessage
      values={{
        link:
          // The next line triggers react/no-multi-comp
          (text: string) => <a>{text}</a>,
      }}
    />
  );
}

I suggest to add an option ignoreNested which ignores any components that aren’t declared as part of the Program node body.

Guida contributor

Direzione di ricerca
Esamina la regola no multi comp nel repository e aggiungi un'opzione 'ignoreNested' che salti i componenti non dichiarati come figli diretti del nodo Program.
Tech stack
javascript
Dominio
developer experiencetooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
ESLint plugin developmentJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70