ManimCommunity/manim

Add linting back to CI

Open

#3 622 ouverte le 16 févr. 2024

Voir sur GitHub
 (5 commentaires) (0 réactions) (0 assignés)Python (1 378 forks)batch import
good first issuehelp wantedtracking

Métriques du dépôt

Stars
 (17 820 stars)
Métriques de merge PR
 (Merge moyen 11j 17h) (4 PRs mergées en 30 j)

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.

Guide contributeur

Direction de recherche
Reportez vous à la PR #3620 pour le contexte. L'objectif est d'ajouter une étape CI utilisant Ruff pour le linting. La configuration peut être effectuée dans pyproject.toml. Vérifiez les fichiers de workflow existants dans .github/workflows pour savoir où ajouter l'étape de linting.
Stack technique
python
Domaine
ci cd
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PythonGitHub Actions
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75