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

Try to fix CI problems #78

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Try to fix CI problems #78

wants to merge 6 commits into from

Conversation

megies
Copy link
Collaborator

@megies megies commented Jan 23, 2018

..and also update install docs.

  • rest framework dropped django 1.9 compat with 3.7.0
  • there's a problem with django parsing geos version. not sure if it's due to conda changing the version number in packaging (then this fix is appropriate) or if the version scheme in geos has changed with newer versions (then another option would be to limit geos version number in Travis

@megies
Copy link
Collaborator Author

megies commented Jan 23, 2018

I think these fixes work but looks like there's also some docs building issue and also one test is failing not sure why.

@megies
Copy link
Collaborator Author

megies commented Apr 12, 2018

@krischer what's the reasoning behind this one? a58e8052d110ca4ae0284e2767bfb55f060ea6ae

It's failing a test that is explicitly using an empty network string (the test data also has an empty network.

@megies
Copy link
Collaborator Author

megies commented Apr 13, 2018

If there's no objections I'm gonna merge this one. Pretty sure that test fails are unrelated and master branch is failing CI anyway.

@megies megies requested a review from krischer April 13, 2018 08:37
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.

1 participant