taichi-dev/taichi

Enable pylint for python/taichi folder

Open

#3107 aperta il 8 ott 2021

Vedi su GitHub
 (2 commenti) (2 reazioni) (1 assegnatario)C++ (2384 fork)batch import
enhancementgood first issuewelcome contribution

Metriche repository

Star
 (28.195 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Inizia esaminando il file `.pylintrc` nella radice del repository. Quindi scegli una regola pylint non selezionata dalla checklist nell'issue (ad esempio 'redefined outer name'). Abilita la regola in `.pylintrc` e correggi tutti gli avvisi risultanti nella directory `python/taichi/`. Assicurati che le modifiche superino i test esistenti prima di creare una pull request. Per descrizioni dettagliate delle regole, fai riferimento alla documentazione di pylint.
Tech stack
python
Dominio
developer experience
Tipo issue
Manutenzione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonGitpylint
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85