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

Test django 2 in travis #233

Merged
merged 5 commits into from
Dec 22, 2017
Merged

Test django 2 in travis #233

merged 5 commits into from
Dec 22, 2017

Conversation

albertyw
Copy link
Member

@albertyw albertyw commented Dec 5, 2017

This adds Django 2 to the list of django versions that travis will test against.

This still requires some work to fix various problems. Also depends on #232 and #231.

@avelis
Copy link
Collaborator

avelis commented Dec 5, 2017

@albertyw I am seeing a couple of issues with TravisCI. It appears it can't seem to download Django 2. The other issue is that the projects url settings don't seem to pass certain versions of Django during the testing matrix. Might have to do conditional url patterns or drop the version that is not ok with the pattern change.

@avelis
Copy link
Collaborator

avelis commented Dec 5, 2017

@albertyw I also remember reading somewhere you wanted to cut a release. The version value is in setup.py. If you want to break that out into a different mechanic that's fine. FYI It is still currently at 1.0.0.

@albertyw albertyw force-pushed the test-django-2 branch 6 times, most recently from 78e55dd to b84b52a Compare December 10, 2017 00:13
@albertyw albertyw mentioned this pull request Dec 16, 2017
@avelis
Copy link
Collaborator

avelis commented Dec 22, 2017

@albertyw Thanks again for working on this and getting the compatibility for Django 2 in this library!

@albertyw albertyw deleted the test-django-2 branch December 23, 2017 13:52
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.

2 participants