Code review comment for lp://staging/~bregma/evemu/lp-731678

Revision history for this message
Chase Douglas (chasedouglas) wrote :

Everything looks sane to me, though I don't know python very well. I'm fine with merging it anyways though. Unit testing is good, creating new bindings is better, and unit testing through the new bindings is awesome :).

The only question I have is whether the python/Makefile.original and python/TODO files were meant to be included. If so, why are they there?

« Back to merge proposal