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 t
On Mon, Sep 9, 2013 at 3:29 PM, Charles R Harris
wrote:
>
>
>
> On Mon, Sep 9, 2013 at 8:12 AM, Richard Hattersley
> 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 and it works well. While we
>
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
FILES=$(gi
On Mon, Sep 9, 2013 at 8:12 AM, Richard Hattersley 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 and it works well. While we
> transition we have an exclusion list (which is gradually getting shorter).
> 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 t
On Sat, Sep 7, 2013 at 7:56 PM, Charles R Harris
wrote:
> 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
> dis
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 operators