search

ZeroQI / Lambda.bundle

Local Media Export, a reverse Plex 'Local Media Assets' agent to export metadata locally
GNU General Public License v3.0
134 stars 18 forks source link

spaces in code #3

Closed ukdtom closed 6 years ago

ukdtom commented 6 years ago

I see you want your code to look pretty, and as such, are inserting spaces to make lines with eq. signs adj. in one row.

That is not allowed according to PEP8, and as such, not recommended.

Strongly suggest, that you install a pep8 plugin to your ide

ZeroQI commented 6 years ago

Pep 8 aim is to improve readability. You are saying it look pretty (ie more readable?) yet not allowed in PEP8... 

Some other good reasons to ignore a particular guideline:
When applying the guideline would make the code less readable, even for someone who is used to reading code that follows this PEP.

Hoewever line length might explain why it often give the wrong code line when errors are detected...

There is no PEP8 checker for Notepad++ which i exclusively develop with

I do value some recommendation of PEP8 but struggle with changing habits as making one single line 5 does not help my readability of the code, but i may code weirdly, i admit...

ukdtom commented 6 years ago

Well, your code, so your call ;)

Just wanted to mention it, in case you didn't knew that.

And FYI: I use this, on both Windows, Mac and Linux https://code.visualstudio.com/

ZeroQI commented 6 years ago

I appreciate your input warmly. I downloaded it and look really nice and seemingly can support github so will give a try. Thanks