jsx-eslint/eslint-plugin-react

Rule Suggestion: Forbid Classes / Classnames

Open

#1193 aperta il 13 mag 2017

Vedi su GitHub
 (7 commenti) (0 reazioni) (0 assegnatari)JavaScript (2797 fork)batch import
acceptedenhancementhelp wantednew rule

Metriche repository

Star
 (8630 star)
Metriche merge PR
 (Merge medio 1g 17h) (3 PR mergiate in 30 g)

Descrizione

We have a font-awesome <Icon> component that handles accessibility nicely. However, it's still really easy for people to write <i className="fa ...">. It would be nice if we could ban a class based on matching a regex. Obviously, it's tricky/impossible to ban every possible use if people are doing things at runtime, but it would be nice if I could say no to a fair number of cases where this comes up.

Guida contributor

Direzione di ricerca
Per implementare questa funzionalità, un contributore dovrebbe esaminare le regole esistenti nella directory `lib/rules/` del repository eslint plugin react per comprendere lo schema di creazione delle regole. La discussione dell'issue suggerisce di aggiungere un'opzione configurabile per abbinare i nomi delle classi tramite regex, simile alla regola `no unknown property`. Il contributore dovrebbe anche rivedere la documentazione di ESLint sulla scrittura di regole personalizzate. I commenti dell'issue (ad esempio, da @ljharb) potrebbero contenere requisiti o vincoli aggiuntivi. La nuova regola dovrebbe essere denominata in modo simile a `no class name regex` e dovrebbe supportare un'opzione che specifichi un pattern regex.
Tech stack
javascriptreact
Dominio
developer experience
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptESLint
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60