Skip to content

Update to work with supported Django versions (2.2-4.1), use GitHub actions for testing - #17

Open
PetrDlouhy wants to merge 12 commits into
greyside:masterfrom
PetrDlouhy:django20
Open

Update to work with supported Django versions (2.2-4.1), use GitHub actions for testing#17
PetrDlouhy wants to merge 12 commits into
greyside:masterfrom
PetrDlouhy:django20

Conversation

@PetrDlouhy

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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