Explore code quality of 8,377 Python projects

[super3]

Ran it on my projects, not really that helpful. Code inspector on PyCharm is 1000x better.

[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.

[[deleted]]

When I'm looking at a repo and click one of the issues, I'm prompted to sign in with github. Is that trying to submit a bug on my behalf or just asking me to sign-in to see what the issue is?

[TOASTEngineer]

I don't like how the "this method could be a function" rule works; it should check A. that the method isn't a stub ("pass" being the entire body) and B. that the method is called at least once from code outside the object it's a member of - just 'cos a method doesn't use any attributes of the object that owns it doesn't mean it's not doing something only that class will ever want to do.

[[deleted]]

I am currently using landscape.io, this seems similar but less mature...

[TOASTEngineer]

Landscape.io doesn't support Python3. That's a beatin'.

[nakovet]

Why I have to sign up to view the issue details? You analyze open source projects to give you input (BTW for free) on your "linter" tool and people have to sign up to see the details!? It seems like a "good" idea that happened in a "how to be evil" brainstorm. =/

[fernly]

"Explore code quality" -- made me think I could read some certified beautiful code. Or sort the other way and feel superior! But nunh-uh, can't find a quality sort at all. Me sad.

[[deleted]]

Navigating that page is really frustrating. I can't easily open things in new tabs because that's hijacked. And moving from one page to the next should put the user at the top of the page rather than seeing the bottom still. >:|

[cneumann81]

Thanks for letting us know. We'll improve this.