-
Notifications
You must be signed in to change notification settings - Fork 1
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
Django 1.9 #27
Django 1.9 #27
Conversation
Not sure, I fully understand. I'll have a look soon. |
- Moved version definition back into __init__.py (instead of setup.py), like the other extensions - Wrapped register() call in __init__.py in try/except to avoid problems when it is called during setup.py - Re-added geokey dependency (>=0.9.3) in requirements.txt as it should now be harmless (at least since geokey v0.9.3; and more specifically this change: ExCiteS/geokey@bd95e68) - Updated .travis.yml to *first* install django (either 1.8.x or 1.9.x depending on the env), and no longer install GeoKey explicitly as it should now be installed by pip as a geokey_sapelli dependency) Signed-off-by: Matthias Stevens <matthias.stevens@gmail.com>
Signed-off-by: Matthias Stevens <matthias.stevens@gmail.com>
This can be merged into master now. If you're okay with it I'll release it as 0.7.2. |
Not really happy with this. The reason I install GeoKey on Travis CI because I want to create a multi scenario for the future GeoKey releases, so that extension would be tested with 0.9, 0.10, 0.11 etc. |
I see. Then I'll put the explicit, env-based, geokey installation back in the travis script. |
Signed-off-by: Matthias Stevens <matthias.stevens@gmail.com>
Done, and everything still works. |
geokey-sapelli cannot use Django 1.8 due to the requirements implementation, which then comes from GeoKey requiring Django===1.8.8.
We can't set Django>=1.8 on GeoKey as proposed, because it will then satisfy Django future releases of Django (let's say 1.10 or 2.0), but not sure if all adjustments will be done on GeoKey side to support those future releases.
Any thoughts?