Skip to content
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

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

PetrDlouhy
Copy link
Contributor

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 around admin.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.

@PetrDlouhy PetrDlouhy changed the title Fix tests for Django20 Fix tests for Django 2.0 and 2.1 Aug 30, 2018
@PetrDlouhy
Copy link
Contributor Author

@SeanHayes I have added also Django 2.1 testing, and it passes.

@PetrDlouhy PetrDlouhy force-pushed the django20 branch 4 times, most recently from be96974 to 6183a1f Compare August 15, 2022 08:57
@PetrDlouhy PetrDlouhy force-pushed the django20 branch 15 times, most recently from 2cbdb89 to 5088dcc Compare August 15, 2022 09:43
@PetrDlouhy PetrDlouhy force-pushed the django20 branch 3 times, most recently from cfdd897 to e579635 Compare August 15, 2022 10:00
@PetrDlouhy PetrDlouhy changed the title Fix tests for Django 2.0 and 2.1 Update to work with supported Django versions (2.2-4.1), use GitHub actions for testing Aug 15, 2022
@PetrDlouhy
Copy link
Contributor Author

@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.

@PetrDlouhy PetrDlouhy mentioned this pull request Aug 19, 2022
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant