4 ms·
I'm "this guy" who "got a bunch of things wrong" :) Style guides are opinionated, and not everyone will agree. I intentionally made the style guide part of a G
by pixelmonkey 11y ago
I'm "this guy" who "got a bunch of things wrong" :)
Style guides are opinionated, and not everyone will agree. I intentionally made the style guide part of a Github repository so I could consider feedback from the community, and I've already incoporated plenty. As I mention below in the thread, I think you have a good point about "args" vs "cards" for your shuffle example.
Re: "The preferred place to break around a binary operator is after the operator, not before it", I think you mis-read this PEP8 suggestion. A binary operator is something like "and" or "+", not the "for" keyword inside a listcomp.
Re: your other disagreements, they amount to style choices of their own. When a style guide picks between two ways of doing things, there will inevitably be detractors, and I am OK with that.
- Walkman 11y agoIf you don't use "if item" properly, it's bad code, not just style choice.