Make TARDIS PEP8 complaint

tardis-sn / tardis

TARDIS - Temperature And Radiative Diffusion In Supernovae

https://tardis-sn.github.io/tardis

202 stars 405 forks source link

Make TARDIS PEP8 complaint #2041

Open epassaro opened 2 years ago

epassaro commented 2 years ago

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.

saiashishanshuman commented 2 years ago

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.

prady0t commented 1 year ago

Hey can I work on this issue ?

wkerzendorf commented 1 year ago

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