ManimCommunity/manim

Add linting back to CI

Ouverte

#3 622 ouverte le 16 févr. 2024

Voir sur GitHubGuide contributeur
 (5 commentaires) (0 réaction) (0 personne assignée)Python (1 378 forks)batch import
good first issuehelp wantedtracking

Métriques du dépôt

Stars
 (17 820 étoiles)
Métriques de merge PR
 (Merge moyen 90j 10h) (31 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
Examinez la configuration actuelle du linting (flake8) et la configuration CI. Référez vous à la PR #3620 pour les modifications précédentes. Configurez Ruff pour le linting (et éventuellement le formatage), puis ajoutez un job CI pour l'exécuter.
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitPython
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70