On Thu, Sep 1, 2011 at 9:00 PM, Trevor Wennblom <trevor(a)well.com> wrote:
given that python has syntactically significant whitespace, i also
try to maintain the convention of indentation with four-spaces.
i've noticed this isn't consistent within the codebase, but does
seem to be the preferred style such as in `lib/galaxy/datatypes/`.
python comes packaged with the script `reindent.py`:
this is recommended practice per PEP 8:
would anyone be opposed to me fixing up the current codebase
to adhere to this? running `reindent.py` on the files is easy enough,
i'm willing to step through the files (`opendiff` / `FileMerge.app`)
and verify no unlikely syntactic changes have occurred. i can also
deliver changes in gradual "chunked" pull requests to ease
current developers getting possibly bit by merge issues.
+1 on correcting any tabs to spaces in the Galaxy Python code.
Doing this in chunked commits makes good sense too - although
if you can get one of the Galaxy team to do this directly it might
Personally I'd like to go further and fix the non-PEP8 white
space in most of the Galaxy Python code, e.g.
function ( argument )
Chatting to some of the Galaxy team at BOSC/ISMB 2011
there is some support for this internally. Again, there are
automated tools to do this.
would anyone be willing to add the appropriate hooks to
the central repository as well?
As long as there are no false positives identified during
the initial tab/space conversion that seems sensible to
prevent new tabs creeping in. But not essential.