search

jayvdb / django-dunder

Django app to attach usable __repr__ and __str__ to any and all Django models, using unmigrated __unicode__ on Python 3
MIT License
2 stars 2 forks source link

Python 2 #12

Open jayvdb opened 4 years ago

jayvdb commented 4 years ago

There are a few minor bugs wrt Python 2, especially with unicode strings, and another just added in 0.3.0 when using the optional round() syntax.

Most importantly with Python 2 and unicode, there are not enough tests that pin-point the bugs, which helps identify how major or minor they are, and helps motivate doing the easy wins, leaving the bigger problems until there is external motivation, which may never eventuate given Python 2 status.

jayvdb commented 4 years ago

And then we get a Python 2 CI failure https://cirrus-ci.com/task/5400585706930176 ; only caused by bug in test logic.