tardis icon indicating copy to clipboard operation
tardis copied to clipboard

Published 20 hours ago • verified publisher icon tardis-sn

Make TARDIS PEP8 complaint

Open epassaro opened this issue 2 years ago • 3 comments

Currently, TARDISis not PEP8 complaint.

To check this, install flake8 and run flake8 tardis.

To achieve this we need to:

  • Configure Flake8 through configuration files.
  • Ensure that Flake8 and Black play nicely together
  • Select appropriate Flake8 plugins, such as pep8-naming (discuss)
  • Make necessary changes to TARDIS code
  • more?

The CI pipeline is ready to check PEP8, just requires removing the if: false from the flake8 job.

epassaro avatar Jun 02 '22 17:06 epassaro

Hey @epassaro, Sai here. So I am a beginner level python user and very new to Open Source. I was hoping to make my contribution to TARDIS and this issue would be a good start for it. Would you please assign this to me and I would try my best to contribute.

saiashishanshuman avatar Jun 09 '22 17:06 saiashishanshuman

Hey can I work on this issue ?

prady0t avatar May 15 '23 20:05 prady0t

As this issue makes a large number of changes in the code - this is not a good issue for beginners.

wkerzendorf avatar May 16 '23 15:05 wkerzendorf