jsx-eslint/eslint-plugin-react

Rule Suggestion: Forbid Classes / Classnames

Open

#1.193 aberto em 13 de mai. de 2017

Ver no GitHub
 (7 comments) (0 reactions) (0 assignees)JavaScript (2.797 forks)batch import
acceptedenhancementhelp wantednew rule

Métricas do repositório

Stars
 (8.630 stars)
Métricas de merge de PR
 (Mesclagem média 1d 17h) (3 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Para implementar esse recurso, um contribuidor deve examinar as regras existentes no diretório `lib/rules/` do repositório eslint plugin react para entender o padrão de criação de regras. A discussão da issue sugere adicionar uma opção configurável para corresponder nomes de classe via regex, semelhante à regra `no unknown property`. O contribuidor também deve revisar a documentação do ESLint sobre como escrever regras personalizadas. Os comentários da issue (por exemplo, de @ljharb) podem conter requisitos ou restrições adicionais. A nova regra deve ser nomeada de forma semelhante a `no class name regex` e deve suportar uma opção que especifique um padrão regex.
Pilha de tecnologia
javascriptreact
Domain
developer experience
Tipo Issue
Funcionalidade
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.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptESLint
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60