taichi-dev/taichi

Enable pylint for python/taichi folder

Open

#3 107 ouverte le 8 oct. 2021

Voir sur GitHub
 (2 commentaires) (2 réactions) (1 assigné)C++ (2 384 forks)batch import
enhancementgood first issuewelcome contribution

Métriques du dépôt

Stars
 (28 195 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Currently if you run pylint python/taichi/ there're a lot of warnings/errors. We'd like to get rid of these and run pylint in CI. Also in .pylintrc we skip a few checkers but it'd be nice to enable some of them back, e.g. import-outside-toplevel.

This work is good for new contributors, and you're welcome to submit one PR per pylint checker/rule.

  • singleton comparison (C0121)
  • import outside top level (C0415)
  • redefined-outer-name (W0621)
  • redefined-builtins (W0622)
  • consider using from import (R0402)
  • use f-string (C0209)
  • no-self-use (R0201)
  • useless-super-delegation (W0235)
  • redefined-outer-name (W0621)
  • unused-variable (W0612)
  • no-else-return (R1705)
  • reimport (W0404)
  • unused-import (W0611)
  • wildcard-import (W0401)

Reference: https://pylint.pycqa.org/en/latest/technical_reference/features.html

Guide contributeur

Direction de recherche
Commencez par examiner le fichier `.pylintrc` à la racine du dépôt. Ensuite, choisissez une règle pylint non cochée dans la liste de l'issue (par exemple 'redefined outer name'). Activez la règle dans `.pylintrc` et corrigez tous les avertissements qui en résultent dans le répertoire `python/taichi/`. Assurez vous que les modifications passent les tests existants avant de créer une pull request. Pour des descriptions détaillées des règles, reportez vous à la documentation de pylint.
Stack technique
python
Domaine
developer experience
Type d'issue
Maintenance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
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
PythonGitpylint
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85