search

ecometrica / django-multisite

Multisite in django — use one Django app to serve multiple domains
BSD 3-Clause "New" or "Revised" License
138 stars 42 forks source link

Upgrade to django 1.11 and improve testing for python/django versions #45

Closed rebkwok closed 7 years ago

rebkwok commented 7 years ago

Upgrade for compatibility with Django 1.11 Remove some try/excepts for Django <1.7, which multisite doesn't support anymore Add pytest, tox and travis integration; test for python versions 2.7 - 3.6 and Django 1.7 - 1.11 Updates to README

Fixes #8, #37, #38, #40, #41, #43

coveralls commented 7 years ago

Coverage Status

Changes Unknown when pulling 7298c6b6daa3cefa7cc5165949686add228277fa on rebkwok:master into on ecometrica:master.

coveralls commented 7 years ago

Coverage Status

Changes Unknown when pulling 5e9219126c88217446d988c86ec56f592b043e7a on rebkwok:master into on ecometrica:master.

rebkwok commented 7 years ago

No, we're still supporting 1.7. Dropped support for <1.7.

On Wed, Apr 26, 2017 at 3:48 PM, jordiecometrica notifications@github.com wrote:

@jordiecometrica commented on this pull request.

In CHANGELOG.rst https://github.com/ecometrica/django-multisite/pull/45#discussion_r113472813 :

@@ -7,6 +7,7 @@ Release Notes

  • Support Django 1.10 (PR #38) and 1.11
  • Support Python 3 +* Remove support for Django <1.7

Just curious, was it hopeless to try to support 1.7?

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/ecometrica/django-multisite/pull/45#pullrequestreview-34864265, or mute the thread https://github.com/notifications/unsubscribe-auth/AGdRBqvuPjAiZ5Sr_ZBKOusH4EOTE8QRks5rz1kzgaJpZM4NA5pX .