mzucker/noteshrink

Multi-version Support?

cathalgarvey opened this issue · 3 comments

Hi Matt, awesome work and a very enjoyable write-up, thank you.

I've looked over the code briefly, and it's very neatly structured and compartmentalised. None of the dependencies look like no-gos for Python3, and the file-handling seems to be compartmentalised by PIL/Pillow.

So, I'm wondering why this is Python2 only? If it's merely personal preference and all it would take are some modernisations from __future__ and some print function calls (or six, if it came to that), would you accept pull requests to make this work on both?

Sure, happy to accept pull requests, especially if you tell me also how to test w/ Python 3 on my machine (I'm a Python 2 user, but it's probably past time to upgrade...)

Think we got this one settled in 0701e13

\o/ Great news, thanks!