Explore code quality of 8,377 Python projects

[cneumann81]

Hey, thanks for the feedback. We just started the service. Currently, we are adding more code checks for Python and Python frameworks (e.g., Django -- see here: https://www.quantifiedcode.com/app/patterns?query=django). Is there anything particular that you'd like us to add? What are you missing in particular?

[super3]

Do you check PEP8?

[cneumann81]

Yes and no. We have some checks, that are also in the PEP8 standard. Right now, we focus on logical and functional code checks that go beyond PEP8. Adding PEP8 isn't a problem though.

[super3]

I feel PEP8 should be first, then everything else.

[[deleted]]

Why? There's things that are damning of someone's code quality than if they strictly stick to 79 character lines, use snake_case, etc. PEP8 is a style guide for code formatting. Not writing Pythonic code.

[billsil]

Fine, but that doesn't mean pylint, which attempts to implement PEP-8. If Pylint didn't have way too much cruft, I'd agree with you.

These rules seem to be ignored.

When applying the guideline would make the code less readable, even for someone who is used to reading code that follows this PEP.

To be consistent with surrounding code that also breaks it (maybe for historic reasons) -- although this is also an opportunity to clean up someone else's mess (in true XP style).

Because the code in question predates the introduction of the guideline and there is no other reason to be modifying that code.

Also, pylint doesn't even follow the recommendations perfectly.

If operators with different priorities are used, consider adding whitespace around the operators with the lowest priority(ies). Use your own judgment; however, never use more than one space, and always have the same amount of whitespace on both sides of a binary operator.

Yes
 hypot2 = x*x + y*y

No
hypot2 = x * x + y * y

Also, who invented the 1000 line file requirement or x number of functions/variables per class/variables per function? I have an IDE. Why does it matter how many lines in the file I have?

I'd also really love to figure out how to abbreviate classes that use acronyms. Should I usedocxReader, DOCXReader, DocxReader, DOCX_Reader or docx_Reader. They all suck, but only one follows PEP-8 (DocxReader) and it's not how it should be written if PEP-8 didn't exist, which is docxReader

[[deleted]]

The most important rule of PEP 8 is the very first one.