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.

Sign up for GitHub

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

Jump to bottom

Allow Django 1.11 #5

Open

nickburlett wants to merge 1 commit into awesto:master from nickburlett:master

nickburlett commented Dec 18, 2017

This adds support for Django 1.11 by updating setup.py.

Fixes #4


          Allow Django 1.11

d47ed19

Author

nickburlett commented Dec 18, 2017

I think the travis build for this pull request fails because the current configuration test configuration isn't correct.

SNI configuration: "Some links may return broken results due to being unable to check the Server Name Indication (SNI) in the returned SSL cert against the hostname in the url requested."
pypy not installed in "python-3.5" config: ERROR: InterpreterNotFound: pypy

I reworked the travis configuration on my fork to:

test more configurations (python2.7 through 3.6, cms 3.4,3.5, django 1.9-1.11)
have 100% code coverage of djangocms-carousel
use python 3.6 for the docs build to avoid the SNI changes

(I don't have pypy configured in the fork though...)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet