search

django-wiki / django-nyt

Notification system for Django with batteries included: Email digests, user settings, JSON API
Apache License 2.0
144 stars 47 forks source link

Fix travis and tests #12

Closed spookylukey closed 9 years ago

spookylukey commented 9 years ago

This does something similar for django-nyt that I did recently for django-wiki. I've separated it out into logical changes which should all be explained by the commit messages. Note that previously the tests were not running on the implied combinations of Django versions - the tests ran only against Django 1.6

benjaoming commented 9 years ago

Thanks for this, and thanks for making nice, tidy, concise, and beautiful things!! :)

spookylukey commented 9 years ago

My pleasure :-)

In this commit I forgot to remove .travis/*, which is now entirely unused I think - perhaps it's easier if you do that directly?

Also, for my https://github.com/django-wiki/django-wiki/pull/359 I need to create Django 1.7 migrations for django-wiki, but the one for wiki.plugins.notifications depends on the migration for django-nyt that I added above (and I think it's probably best if I don't try to get around that with manually hacking the migration). I think the easiest way to address this would be to release an updated version django-nyt to PyPI - would that be possible?

benjaoming commented 9 years ago

@spookylukey

Yes, that's absolutely possible... I'll release it now...

benjaoming commented 9 years ago

New release! Let me know if it works as expected...