On Sat, Sep 7, 2013 at 7:56 PM, Charles R Harris
charlesr.har...@gmail.comwrote:
Hi All,
I've been doing some PEP8 work using autopep8. One problem that has turned
up is that the default behavior of autopep8 is version dependent. I'd like
to put a script in numpy tools that runs autopep8
Something we have done in matplotlib is that we have made PEP8 a part of
the tests.
In Iris and Cartopy we've also done this and it works well. While we
transition we have an exclusion list (which is gradually getting shorter).
We've had mixed experiences with automatic reformatting, so prefer
On Mon, Sep 9, 2013 at 8:12 AM, Richard Hattersley rhatters...@gmail.comwrote:
Something we have done in matplotlib is that we have made PEP8 a part of
the tests.
In Iris and Cartopy we've also done this and it works well. While we
transition we have an exclusion list (which is gradually
I think a good solution would to use add a git_hooks directory with a
pre-commit git hook along with an git hook installation script. And a note
should be added to DEV_README.txt suggesting installing the git hooks for
pep8 compatibility. I personally use this as a pre-commit
#!/bin/sh
On Mon, Sep 9, 2013 at 3:29 PM, Charles R Harris
charlesr.har...@gmail.com wrote:
On Mon, Sep 9, 2013 at 8:12 AM, Richard Hattersley rhatters...@gmail.com
wrote:
Something we have done in matplotlib is that we have made PEP8 a part of
the tests.
In Iris and Cartopy we've also done this
I made a PR request about supplying a git hooks framework at
https://github.com/numpy/numpy/pull/200. I asked for it to be closed
because I couldn't easily figure our how to handle x-platform issues.
If you have an answer, what I was working on might be a start. But
your script is an example of
Hi All,
I've been doing some PEP8 work using autopep8. One problem that has turned
up is that the default behavior of autopep8 is version dependent. I'd like
to put a script in numpy tools that runs autopep8 with some features
disabled, namely
1. E226 -- puts spaces around arithmetic