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

Make Django2.0 compatible: use simple_tag, closes #69 #70

Closed
wants to merge 1 commit into from
Closed

Make Django2.0 compatible: use simple_tag, closes #69 #70

wants to merge 1 commit into from

Conversation

rixx
Copy link
Contributor

@rixx rixx commented Dec 2, 2017

I saw the previous discussion on #65, and while I agree that dropping versions and other major changes should have to wait, I'm anxious to move to Django 2.0, and as long as this involves only a few corrected imports, I'd be really happy if you'd decide to merge this PR and make a patch release.

@rixx rixx mentioned this pull request Dec 2, 2017
4 tasks
@rixx
Copy link
Contributor Author

rixx commented Dec 2, 2017

I don't quite see how the occurring pypy errors (occurring only with pypy between Django 1.5 and 1.7) can be this PR's fault, but if you point me in the right direction, I'll gladly tackle them.

@riconnon
Copy link

riconnon commented Dec 6, 2017

@rixx Django 1.5-1.7 are all no longer supported, and were never supported by Django project with python version 3.5 (which this pypy3 is equivalent to). Personally, I think these test environments should just be removed.

@rixx
Copy link
Contributor Author

rixx commented Dec 6, 2017

Sounds plausible enough. @dfunckt, what do you think –could you merge this PR and provide a patch release?

@riconnon riconnon mentioned this pull request Dec 6, 2017
@dfunckt
Copy link
Owner

dfunckt commented Dec 12, 2017

d2a8831 adds Django 2.0 support. Thanks for the PR!

@dfunckt dfunckt closed this Dec 12, 2017
@rixx rixx deleted the django2.0 branch December 12, 2017 23:55
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.

None yet

3 participants