-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update to work with supported Django versions (2.2-4.1), use GitHub actions for testing #17
base: master
Are you sure you want to change the base?
Conversation
…heck if it breaks something)
@SeanHayes I have added also Django 2.1 testing, and it passes. |
be96974
to
6183a1f
Compare
2cbdb89
to
5088dcc
Compare
cfdd897
to
e579635
Compare
@SeanHayes I have updated the whole project pass tests in currently supported Django versions and migrated to GitHub actions. I would like to update this project and add some more features. Can you please give me access to this repository? I already have access to PyPI and I can change the home repository there, but I would like to keep the issues and also credit you as creator of this project. |
This fixes tests in Django versions up to 2.0. It also contains few code fixes and cleanings.
It contains one change, that could potentially break something - the removal of bare
try
-except
aroundadmin.autodiscover()
. I don't know, what was purpose of that, but it could overshadow some admin problems. If there is reason for that, we should catch only certain error classes.