-
Notifications
You must be signed in to change notification settings - Fork 6
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
Support reverse from Django 2.0 #7
Conversation
73d1597
to
ed7cb31
Compare
@gavinwahl friendly ping. Could you review this PR? |
@gavinwahl could you reviewing my PR? |
Also pinging on this, the branch works well for us too. |
@@ -8,3 +8,4 @@ pep8.txt | |||
dist/ | |||
build/ | |||
.tox | |||
venv/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't find any environment directory name and added venv
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this necessary? Doesn't tox support multiple python versions?
@gavinwahl tox was not working with travis ci please see https://travis-ci.org/fusionbox/django-absoluteuri/builds/340116977?utm_source=github_status&utm_medium=notification |
@gavinwahl do you have a time for review this PR and bump version? kind regards! |
I believe it is time to approve this. What do you think? Let's update python and django versions? |
fixed in fc377ca |
The django.core.urlresolvers module is removed in favor of its new location, django.urls.
Closes #8