Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Improvements to old TODOs #125

Merged
merged 8 commits into from Apr 7, 2013

Conversation

Projects
None yet
2 participants
Owner

pwaller commented Apr 7, 2013

xref #121

Owner

pwaller commented Apr 7, 2013

Nice big negative line count in this one. Care to take a look, @cdeil? This one is ready, it's as much as I will do right now, though I will enhance any text if you ask about it inline. I hope you (or anyone) might be able to look at them and make sense of what is missing.

Owner

cdeil commented Apr 7, 2013

Can you fix the PEP8 (see travis-ci)?

Owner

pwaller commented Apr 7, 2013

@cdeil, fixed. Any other comments? If not, merge at will.

@cdeil cdeil added a commit that referenced this pull request Apr 7, 2013

@cdeil cdeil Merge pull request #125 from pwaller/121-fix-todos
Improvements to old TODOs
6d0355b

@cdeil cdeil merged commit 6d0355b into rootpy:master Apr 7, 2013

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment