Mailman 3 January 2015 - code-quality

[ANN] Pylint 1.4.1 / Astroid 1.3.3 released
by Claudiu Popa Jan. 16, 2015

Jan. 16, 2015

Hello, It's my pleasure to announce the release of both Pylint 1.4.1 and Astroid 1.3.3 respectively. The following bug fixes and features made their way into Astroid 1.3.3: * Restore file_stream to a property, but deprecate it in favour of the newly added method Module.stream. * Add inference tips for 'tuple', 'list', 'dict' and 'set' builtins. * Add brain definition for most string and unicode methods * Add a new method to Class nodes, 'mro', for obtaining the the method resolution order of the class. * Add brain tips for six.moves. Closes issue #63. * .slots() can contain unicode strings on Python 2. * Add inference tips for nose.tools. The complete list of changes can be seen here: https://bitbucket.org/logilab/astroid/src/15cff6016b793bb64c92d04acc4da001a… For Pylint, we have the following fixes and features: * Add a new JSON reporter, usable through -f flag. * Add a new warning, 'redundant-unittest-assert', emitted when using unittest's methods assertTrue and assertFalse with constant value as argument. * Check the return of properties when checking for not-callable. Closes issue #406. * Warn about using the input() or round() built-ins for Python 3. Closes issue #411. * Use a mro traversal for finding abstract methods. Closes issue #415. * Fix a false positive on Python 2 for raising-bad-type, when raising tuples in the form 'raise (ZeroDivisionError, None)'. * Fix a false positive with invalid-slots-objects, where the slot entry was an unicode string on Python 2. Closes issue #421. The complete list of changes can be found here: https://bitbucket.org/logilab/pylint/src/02eabae6b72569876e7104db7724ad26a7… If you find any bugs, don't hesitate to open a new issue on our issue tracker (https://bitbucket.org/logilab/pylint/) Enjoy!

1 0

Getting Flake8 to report syntax errors.
by Kannan Goundan Jan. 15, 2015

Jan. 15, 2015

Sample input file "test.py": def f(x) print x Pyflakes reports the syntax error: $ pyflakes test.py test.py:1:9: invalid syntax def f(x) ^ $ But Flake8 doesn't: $ flake8 test.py $ Is this intentional? Is there a way to get Flake8 to report syntax errors? $ flake8 --version 2.3.0 (pep8: 1.5.7, pyflakes: 0.8.1, mccabe: 0.3) CPython 2.7.5 on Darwin $ pyflakes --version 0.8.1

2 1

[feature request] .gitignore option for flake8
by Austin Macdonald Jan. 12, 2015

Jan. 12, 2015

I would like if flake8 could ignore all files that match the .gitignore. This would be helpful since it is not ideal to list things that are specific to a specific dev environment in automatic test runners. Thanks, Austin

4 3

flake8 config file usage
by Jason Killen Jan. 9, 2015

Jan. 9, 2015

I've just started using flake8. I'm using 2.2.5 (pep8: 1.5.7, pyflakes: 0.8.1, mccabe: 0.3, flake8-debugger: 1.3.2) CPython 2.7.1 on Darwin Config files don't seem to be being read in. I've tried defining it (config.cfg) using --config, putting it in ., putting it in everything up from ., and putting it in ~/.config/flake8. No matter what I try flake8 doesn't seem to be picking them up. Using --ignore works fine. Here's my config file: [flake8] max-line-length = 200 # E251 unexpected spaces around keyword / parameter equals # E711 comparison to None should be if cond is not None # E501 line too long (203 > 200 characters) # E303 too many blank lines (2) # E712 comparison to True should be 'if cond is not True:' or 'if not cond:' # E302 expected 2 blank lines, found 1 # E226 missing whitespace around arithmetic operator # W291 trailing whitespace # W293 blank line contains whitespace # W391 blank line at end of file # E111 indentation is not a multiple of four # E702 multiple statements on one line (semicolon) ignore = E251,E711,E501,E303,E712,E302,E226,W291,W293,W391,E111,E702,E265 Help please? -- Jason Killen jsnklln(a)gmail.com Pain is inevitable, misery is optional.

2 6

pylint: Writing checker to check method calls
by Florian Bruhin Jan. 8, 2015

Jan. 8, 2015

Hi! I'm trying to write a checker for pylint which checks if certain callables (anything passed to the .connect() method of a pyqtSignal, i.e. Qt slots) are decorated with @pyqtSlot. However I'm already stuck trying to implement a visit_callfunc which actually sees the call. My checker does this: def visit_callfunc(self, node): if hasattr(node, 'func'): if getattr(node.func, 'name', None) == 'connect': import pdb; pdb.set_trace() (full source: http://paste.the-compiler.org/view/32c1b8df) This is the code I'm testing with: from PyQt5.QtCore import QObject, pyqtSignal class Foo(QObject): sig = pyqtSignal() def __init__(self): self.sig.connect(func) def func(): pass (paste: http://paste.the-compiler.org/view/1970b9b8) However, the breakpoint never hits. Why is that? Florian -- http://www.the-compiler.org | me(a)the-compiler.org (Mail/XMPP) GPG: 916E B0C8 FD55 A072 | http://the-compiler.org/pubkey.asc I love long mails! | http://email.is-not-s.ms/

1 0

[Announcement] [mccabe] New way to run mccabe as a standalone script
by Ian Cordasco Jan. 6, 2015

Jan. 6, 2015

Hey all, Ever since mccabe has become it's own package on PyPI, you've been able to run it independently by executing python -m mccabe {opts} Since most of mccabe's downloads are via Flake8, the project has resisted adding an entry point to make an easy-to-access console script. I'm a thoroughly curious person though, so I threw together a package on PyPI - mccabe-console-script - to provide that functionality. If the download count demonstrates that this is useful, I'll propose updating the mccabe package to include the extra lines that make this possible. The repository is on GitLab [1] and mirrored to GitHub [2] with the source code. Cheers, Ian [1]: https://gitlab.com/pycqa/mccabe-console-script [2]: https://github.com/PyCQA/mccabe-console-script

1 1

Pylint custom transformer: suppress some error messages
by Alexandr Presniakov Jan. 5, 2015

Jan. 5, 2015

Hello! I'm implementing pylint plugin for web2py: it executes user code in a special environment populated with predefined objects. My transformer adds imports for pylint not to complain about undefined objects. But after that pylint complains about unused-wildcard-import. Is there any way to suppress some messages or at least to add some '#pylint: disable=...' line to module from the transformer? Thanks in advance, Alex

2 1