astral-sh/ruff

Make logger object name patterns configurable

Aberta

#12.613 aberto em 1 de ago. de 2024

Ver no GitHubGuia do colaborador
 (6 comentários) (3 reações) (0 responsável)Rust (2.088 forks)batch import
help wantedtype-inference

Métricas do repositório

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

Description

Currently, Ruff uses some hard-coded name "patterns" to determine whether a module-local object might be a logger (for the purposes of e.g. flake8-logging-format G rules).

https://github.com/astral-sh/ruff/blob/fc16d8d04d86aa94a8aac14bbb87089b64d443a1/crates/ruff_python_semantic/src/analyze/logging.rs#L60-L74

We've found that we have some loggers called _LOG which aren't matched, and I was disappointed to find that these aren't configurable. Maybe this should be a setting list of regular expressions or globs or something like that?

Guia do colaborador

Direção de pesquisa
Examine os padrões de nome de logger atualmente codificados em crates/ruff python semantic/src/analyze/logging.rs, depois adicione uma nova opção de configuração (por exemplo, no pyproject.toml) para permitir que os usuários especifiquem padrões personalizados e atualize a lógica para usar esses padrões.
Pilha de tecnologia
rustpython
Domain
developer experiencecli
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Rust
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60