astral-sh/ruff

`N999` (`invalid-module-name`) mixes correctness and stylistic recommendations

Aberta

#12.475 aberto em 23 de jul. de 2024

Ver no GitHubGuia do colaborador
 (11 comentários) (2 reações) (0 responsável)Rust (2.088 forks)batch import
help wantedrule

Métricas do repositório

Stars
 (47.527 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Not only does rule N999 check that the PEP8 convention is respected, but also checks that the module name is syntactically correct. The latter is not part of PEP8. This is a problem for me currently working on "old non linted" code. I have to ignore this rule because of some modules whose names are syntactically correct but sometimes with capital letters. I think a E rule is missing that would only include the syntax checking of N999. (If I were you I would also remove that part from N999 for consistency)

Guia do colaborador

Direção de pesquisa
Investigue como as regras de lint são definidas no codebase do Ruff, localize a implementação da regra N999, entenda como ela combina verificações de estilo PEP8 com a validação de sintaxe do nome do módulo e pesquise como criar uma nova regra E separada para a verificação de sintaxe.
Pilha de tecnologia
rustpython
Domain
clideveloper 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.
1-2 dias
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
RustPython
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60