◐ Shell
clean mode source ↗

Travis CI: 'sudo' tag is now deprecated in Travis by cclauss · Pull Request #825 · gitpython-developers/GitPython

jeking3

@@ -9,10 +9,8 @@ matrix:
include:
- python: 3.7
dist: xenial

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You might be able to move dist: xenial out one level and define it globally. If so, you can move the python declarations out to the global list and eliminate the matrix include section all together.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can confirm that both solutions work. While xenial is generally available, it is not the Travis default distro so I usually take the more conservative route of only using it where it is needed.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#831 eliminates one test when run on xenial so I think it is best to run more tests on Pythons that are compatible with the Travis CI default distro and fewer tests only on Python 3.7 and nightly.

@codecov-io

@Byron Byron mentioned this pull request

Jan 20, 2019