search

hayd / pep8radius

PEP8 clean only the parts of the files touched since the last commit, a previous commit or (the merge-base of) a branch.
MIT License
183 stars 9 forks source link

use cdiff for colored diffs #60

Closed hayd closed 10 years ago

hayd commented 10 years ago

ATM I've rolled my own diff printer, should probably use cdiff - which also allows for paging and side-by-side which would be pretty neat.

Note: I don't think cdiff existed when pep8radius started (I recall looking), I had been considering breaking out my diff printer to a sep lib...

hayd commented 10 years ago

As mentioned on a related autopep8 issue you can do:

pep8radius master --diff --no-color | cdiff

which is really neat. Note: cdiff may not support windows.

hayd commented 10 years ago

I think maybe just have this as an example in README. Perhaps also show as a git alias:

[alias]
  rad = !pep8radius master --diff --no-color $@ | cdiff --side-by-side

allows git rad and git rad -i, which are really neat (requires cdiff).

hayd commented 10 years ago

screen shot 2014-09-13 at 01 06 27