-
Notifications
You must be signed in to change notification settings - Fork 21
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
Travis CI broken / useless #19
Comments
Hey @mtulio, as I badly need to have working tests, I invested some time today... Could you have a look at https://github.com/poikilotherm/glpi-sdk-python/tree/tox_docker ? Build status and logs can be seen on Travis: https://travis-ci.org/poikilotherm/glpi-sdk-python Thanks! |
@augustoferronato could you comment on this? @mtulio seems to be busy. |
Sorry for the delay, we are working on to solve all issues |
Make the PR |
@augustoferronato it is not yet in master branch, we need to test the referenced PR (#27) . I just reopened here. =) |
@mtulio ok, no problem. |
Hi @mtulio,
with greatest respect for your work: did you notice the Travis builds are more or less useless?
Some don't build because of faulty python usage, the rest declares things as built but actually just skips the tests because there is no real GLPI to test against.
Maybe it would be an option to use some Docker containers for a testing GLPI instance and containers for the python versions started by the Travis job?
Cheers
Oliver
The text was updated successfully, but these errors were encountered: