taichi-dev/taichi

Enable pylint for python/taichi folder

Open

#3.107 geöffnet am 8. Okt. 2021

Auf GitHub ansehen
 (2 Kommentare) (2 Reaktionen) (1 zugewiesene Person)C++ (2.384 Forks)batch import
enhancementgood first issuewelcome contribution

Repository-Metriken

Stars
 (28.195 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Beginnen Sie mit der Überprüfung der `.pylintrc` Datei im Stammverzeichnis des Repositorys. Wählen Sie dann eine nicht aktivierte pylint Regel aus der Checkliste im Issue (z. B. 'redefined outer name'). Aktivieren Sie die Regel in `.pylintrc` und beheben Sie alle resultierenden Warnungen im Verzeichnis `python/taichi/`. Stellen Sie sicher, dass die Änderungen bestehende Tests bestehen, bevor Sie einen Pull Request erstellen. Weitere Informationen zu den Regeln finden Sie in der pylint Dokumentation.
Tech Stack
python
Domain
developer experience
Issue Type
Wartung
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
PythonGitpylint
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85