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

Upgrade to Django>=2.2 #44

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

Conversation

benmurden
Copy link
Contributor

  • Add tox and pytest for easier testing and coverage reports
    • django-coverage is no longer maintained
  • Make new migrations
  • Add on_delete=models.PROTECT to foreign keys.
  • Remove compatibility code for Django<2.2.
  • Update importlib from Django version to Python standard lib.
  • Update urllib2 to Python 3 standard lib.
  • Update encoding for Mixpanel task.
  • Change print statements to Python 3 style.
  • Change test files to test_* naming convention.
  • Add manage.py for easier testing with tox and pytest.
  • Remove django-celery. No longer needed with modern Celery.
  • Remove mock and use Python 3 unittest.mock.
  • Update statsd implementation and tests.
  • Update management command base class.
  • Add timezone support. Replaces Update to add timezone support. #42.
  • Update test settings.
  • Exclude tests from package.
  • Update test requirements.

@benmurden benmurden marked this pull request as ready for review June 29, 2020 10:11
@benmurden
Copy link
Contributor Author

@frankwiles Have you had a chance to look at this? Anything I can help with?

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