search

Tarskin / MassyTools

A data processing tool for targeted high-throughput MALDI-MS data extraction.
Apache License 2.0
3 stars 4 forks source link

Compliance with PEP8 #3

Closed iugrina closed 6 years ago

iugrina commented 9 years ago

PEP8 is the official Style Guide for Python Code and should be used (as much as possible) to ease the reading and refactoring of code.

I can adjust the code to conform to PEP8 if @Tarskin finds it a good idea.

Tarskin commented 9 years ago

I will have a look at it myself as well (to see if there are any big issues).I thought that I followed it (in general) but feel free to have a go at it.

iugrina commented 9 years ago

Is it ok to go with spaces instead of tabs? If you like tabs more, that's also ok :)

Tarskin commented 9 years ago

I prefer tabs to be honest ;)

On Wed, Jul 8, 2015 at 11:50 PM, Ivo Ugrina notifications@github.com wrote:

Is it ok to go with spaces instead of tabs? If you like tabs more, that's also ok :)

— Reply to this email directly or view it on GitHub https://github.com/Tarskin/MassyTools/issues/3#issuecomment-119741349.

iugrina commented 9 years ago

Ok.

I've improved some parts of the code to be more compliant with PEP8 but I also introduced some movements with inline comments :( (related to the usage of tabs). I'll correct that in the final step of adjustment to PEP8.