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

Travis: Combine jobs, flake8 has been moved to semaphore #13265

Merged
merged 1 commit into from Jan 15, 2020

Conversation

@WickedShell
Copy link
Contributor

WickedShell commented Jan 14, 2020

This does a couple of things to improve CI times:

  • The SITL and unit-test tasks are now both done with clang-7, and are done in the same instance. We spent more time spooling up/installing tooling then we did doing either of these tasks, so we can save a far amount of machine time here.
  • Flake8 is now run on SemaphoreCI instead, the only downside here is due to python versioning nightmares I ended up using Python 3.6 here instead of Python 3.7.
  • Not visible in the PR, but I also advanced the semaphore platform, which gives us a slightly newer set of dependencies to work from.

The net effect is travis should be a lot more responsive and complete it's tasks quicker/PR's should get less backlogged.

@WickedShell WickedShell added the CI label Jan 14, 2020
@WickedShell WickedShell requested a review from OXINARF Jan 14, 2020
@peterbarker peterbarker merged commit 29d31ac into ArduPilot:master Jan 15, 2020
4 checks passed
4 checks passed
ArduPilot.ardupilot Build #20200114.21 succeeded
Details
ArduPilot.ardupilot (Cygwin SITL build) Cygwin SITL build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
semaphoreci The build passed on Semaphore.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.