Python3 support? - Githubissues

tarmstrong / nbdiff

A diffing and merging tool for the IPython Notebook.

http://nbdiff.org

MIT License

212 stars 28 forks source link

Python3 support? #215

Open rgaiacs opened 10 years ago

rgaiacs commented 10 years ago

Thanks for supply this great tool for the community.

When I try to install it I got some warnings related with the use of print statement instead of print function since I'm using Python3 and didn't notice any information about it in README. There is any plan to supply a Python3 compatible version?

tarmstrong commented 10 years ago

Thanks! It shouldn't be too hard to support python3. I'll see if I can find some time to port it, but pull requests are welcome.

rgaiacs commented 10 years ago

When using 2to3 there are some changes that I don't know the best way to keep it compatible with Python2:

replace urllib2 with urllib.request at nbdiff/server/command/MergeURLCommand.py and nbdiff/server/command/DiffURLCommand.py,
replace __nonzero__ with __bool__ at nbdiff/server/database/__init__.py,
keys() behavior at nbdiff/adapter/git_adapter.py.

asmeurer commented 10 years ago

Just use if PY3 constructs. For nonzero, just keep both methods, with one equals the other.