jsx-eslint/eslint-plugin-react

Rule Suggestion: Forbid Classes / Classnames

Open

#1 193 ouverte le 13 mai 2017

Voir sur GitHub
 (7 commentaires) (0 réactions) (0 assignés)JavaScript (2 797 forks)batch import
acceptedenhancementhelp wantednew rule

Métriques du dépôt

Stars
 (8 630 stars)
Métriques de merge PR
 (Merge moyen 1j 17h) (3 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Pour implémenter cette fonctionnalité, un contributeur doit examiner les règles existantes dans le répertoire `lib/rules/` du dépôt eslint plugin react pour comprendre le modèle de création de règles. La discussion de l'issue suggère d'ajouter une option configurable pour faire correspondre les noms de classe via une expression régulière, similaire à la règle `no unknown property`. Le contributeur doit également consulter la documentation d'ESLint sur l'écriture de règles personnalisées. Les commentaires de l'issue (par exemple, de @ljharb) peuvent contenir des exigences ou des contraintes supplémentaires. La nouvelle règle doit être nommée de manière similaire à `no class name regex` et doit prendre en charge une option spécifiant un motif d'expression régulière.
Stack technique
javascriptreact
Domaine
developer experience
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptESLint
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60