crickets-and-comb/shared

Remove common noqas.

Open

#41 aberto em 25 de mar. de 2025

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Makefile (0 forks)auto 404
bugenhancementhelp wanted

Métricas do repositório

Stars
 (2 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Throughout our codebases, we ignore some common linting errors that are raised due to a mismatch between our formatting tools and our linting tools. Examples include E712 and E203. As a result, we have # noqa: E712 for example spread all over, and we often need to add these noqas to new code.

We should remove the need for these noqas in one of two ways:

  1. Add the class of linting error to the shared config files (usually the flake8 file).
  2. Update our formatting tool to modify the code to meet the standard.
  3. We may no longer need them in the first place due to updates in the meantime, so try first just removing the noqa altogether.

We should then remove the noqas wherever they exist in consuming repos.

In some cases, we may want to always fail to force developers to intentionally seek a solution or ignore it.

Guia do colaborador

Direção de pesquisa
Identifique todos os comentários `# noqa` no código, especialmente aqueles relacionados a E712 e E203. Verifique se a regra de linting pode ser adicionada ao arquivo de configuração compartilhado do flake8 para ignorá la globalmente. Se possível, adicione a regra à configuração e remova os comentários noqa. Alternativamente, certifique se de que a ferramenta de formatação (ex: Black) esteja configurada para produzir código que passe no linting. Teste executando o linter e as ferramentas de formatação. Se não forem mais necessários, simplesmente remova os comentários noqa e verifique se o linter passa.
Pilha de tecnologia
python
Domain
backenddeveloper experience
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
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.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PythonGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70