code-quality April 2018

code-quality@python.org

10 participants
5 discussions

C0326 not picking up whitespace before open bracket with print
by Mark McDowall 25 Apr '18

25 Apr '18

Hi, While using pylint I have found that having a space between 'print' and the open bracket is allowed. Based on the PEP8 pet peeves I would have assumed that 'print ("hello!")' would have raised the C0326 warning. Is there a reason why this is not the case? I can't seem to find a post anywhere talking about allowing this. Thank you, Mark

2 1

Some other additions to PyCQA possibly
by Carl Crowder 21 Apr '18

21 Apr '18

Hi there, There are a few other things that came out of the ashes of Landscape.io that may or may not make sense to move into PyCQA too: 1) pylint-celery - this one I don’t think anyone uses and it isn’t very useful anyway 2) pylint-plugin-utils - this is used by pylint-django and pylint-celery as far as I know, I’m not sure how useful it is outside of that 3) prospector - this is the bigger one, lots of people use this, but I can understand that it might not be in the same state of mind as the rest of the PyCQA code :-) Let me know what you think either way! Carl

5 8

Re: [code-quality] [feature request] Unused variable warning for "except Exception as e"
by Margaret Slowikowska 19 Apr '18

19 Apr '18

Sent from my iPhone

1 0

Catching multiple return types
by Skip Montanaro 19 Apr '18

19 Apr '18

Is there a static checker which will catch multiple return types? Consider this dumb example: def func(a, b, c): if c: return (a, b) return 0 I thought pylint had a checker for that, but running with --rcfile=/dev/null didn't complain about the different types being returned. Is there perhaps a checker for that which isn't enabled by default? Or another static tool which will catch it? Thx, Skip

3 3

[feature request] Unused variable warning for "except Exception as e"
by Shane Creighton-Young 16 Apr '18

16 Apr '18

Hello flake8! I would have expected pyflakes to emit an unused variables warning for "e" on the following code snippet: class ModuleError(Exception): pass try: 1 / 0 except ZeroDivisionError as e: raise ModuleError() Curious if this check can be added. Thank you! -- Shane Creighton-Young shane(a)goubiq.com

2 2

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

code-quality April 2018