ManimCommunity/manim

Add linting back to CI

Open

#3.622 aberto em 16 de fev. de 2024

Ver no GitHub
 (5 comments) (0 reactions) (0 assignees)Python (1.378 forks)batch import
good first issuehelp wantedtracking

Métricas do repositório

Stars
 (17.820 stars)
Métricas de merge de PR
 (Mesclagem média 11d 17h) (4 fundiu PRs em 30d)

Description

Following https://github.com/ManimCommunity/manim/pull/3620, would be great to enforce linting on CI.

I propose using Ruff instead of flake8, it would avoid having to install a bunch of plugins and pyproject.toml configuration is supported.

Format could be included as well if the diff isn't to big.

Guia do colaborador

Direção de pesquisa
Consulte a PR #3620 para contexto. O objetivo é adicionar uma etapa de CI usando o Ruff para linting. A configuração pode ser feita no pyproject.toml. Verifique os arquivos de fluxo de trabalho existentes em .github/workflows para saber onde adicionar a etapa de linting.
Pilha de tecnologia
python
Domain
ci cd
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.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
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
PythonGitHub Actions
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75