Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > comp.lang.python > #108299
| From | Stephen Hansen <me@ixokai.io> |
|---|---|
| Newsgroups | comp.lang.python |
| Subject | Re: Pylint prefers list comprehension over filter... |
| Date | 2016-05-07 14:35 -0700 |
| Message-ID | <mailman.476.1462656953.32212.python-list@python.org> (permalink) |
| References | <572BF2BF.6000000@icloud.com> <1462498631.232041.599637409.25D91C08@webmail.messagingengine.com> <572E3F53.5010703@icloud.com> <1462656950.727725.601099753.30D90B95@webmail.messagingengine.com> |
On Sat, May 7, 2016, at 12:17 PM, Christopher Reimer wrote: > On 5/5/2016 6:37 PM, Stephen Hansen wrote: > > On Thu, May 5, 2016, at 06:26 PM, Christopher Reimer wrote: > >> Which is one is correct (Pythonic)? Or does it matter? > > First, pylint is somewhat opinionated, and its default options shouldn't > > be taken as gospel. There's no correct: filter is fine. > > Since the code I'm working on is resume fodder (i.e., "Yes, I code in > Python! Check out my chess engine code on GitHub!"), I want it to be as > Pythonic and PEP8-compliant as possible. That includes scoring 10/10 > with pylint. Never know when an asshat hiring manager would reject my > resume out of hand because my code fell short with pylint. > > For my purposes, I'm using the list comprehension over filter to keep > pylint happy. Bear in mind, when I say, "Pylint is opinionated", I mean the tool -- especially in its default configuration -- has its own opinion of what is good style, and *I think its wrong on a number of points*. Its fine to use, but I'd read over PEP8 (the document, not the tool) and apply style guide recommendations thoughtfully, not mechanically. -- Stephen Hansen m e @ i x o k a i . i o
Back to comp.lang.python | Previous | Next — Next in thread | Find similar | Unroll thread
Re: Pylint prefers list comprehension over filter... Stephen Hansen <me@ixokai.io> - 2016-05-07 14:35 -0700 Re: Pylint prefers list comprehension over filter... Steven D'Aprano <steve@pearwood.info> - 2016-05-08 12:53 +1000
csiph-web