Skip to content

Commit 9f88641

Browse files
committed
Auto merge of #3731 - phansch:travis, r=phansch
Travis: Don't run integration tests on every PR commit This does not save Clippy any time but it makes sure that the concurrent build limit is not reached as quickly for the `rust-lang` Travis account. I can't create a permalink to the discussion somehow, so here's an excerpt from the Infra channel: ``` [11:57 PM] pietroalbini: there is a clippy build (20 jobs) and a packed_simd one (42 builders) and a rustc one which isn't scheduling atm [11:58 PM] pietroalbini: I don't think there is a way to prioritize rustc builds in the queue on travis, right? [12:22 AM] alexcrichton: pietro: I don't think so no [12:22 AM] alexcrichton: If it's a problem we should cull builds on other projects [12:22 AM] alexcrichton: The rust repo is the #1 priority ``` Since the integration tests are rarely failing these days, I think it's fine to not run them on every commit. If needed, it's also still possible to do a complete test run with `@bors try`.
2 parents 2972d54 + 69d96c7 commit 9f88641

File tree

1 file changed

+19
-14
lines changed

1 file changed

+19
-14
lines changed

.travis.yml

+19-14
Original file line numberDiff line numberDiff line change
@@ -39,40 +39,45 @@ install:
3939
matrix:
4040
fast_finish: true
4141
include:
42+
# Builds that are executed for every PR
4243
- os: osx # run base tests on both platforms
4344
env: BASE_TESTS=true
4445
- os: linux
4546
env: BASE_TESTS=true
4647
- os: windows
4748
env: CARGO_INCREMENTAL=0 BASE_TESTS=true
49+
50+
# Builds that are only executed when a PR is r+ed or a try build is started
51+
# We don't want to run these always because they go towards
52+
# the build limit within the Travis rust-lang account.
4853
- env: INTEGRATION=rust-lang/cargo
49-
if: repo =~ /^rust-lang\/rust-clippy$/
54+
if: repo =~ /^rust-lang\/rust-clippy$/ AND branch IN (auto, try)
5055
- env: INTEGRATION=rust-random/rand
51-
if: repo =~ /^rust-lang\/rust-clippy$/
56+
if: repo =~ /^rust-lang\/rust-clippy$/ AND branch IN (auto, try)
5257
- env: INTEGRATION=rust-lang-nursery/stdsimd
53-
if: repo =~ /^rust-lang\/rust-clippy$/
58+
if: repo =~ /^rust-lang\/rust-clippy$/ AND branch IN (auto, try)
5459
- env: INTEGRATION=rust-lang/rustfmt
55-
if: repo =~ /^rust-lang\/rust-clippy$/
60+
if: repo =~ /^rust-lang\/rust-clippy$/ AND branch IN (auto, try)
5661
- env: INTEGRATION=rust-lang-nursery/futures-rs
57-
if: repo =~ /^rust-lang\/rust-clippy$/
62+
if: repo =~ /^rust-lang\/rust-clippy$/ AND branch IN (auto, try)
5863
- env: INTEGRATION=rust-lang-nursery/failure
59-
if: repo =~ /^rust-lang\/rust-clippy$/
64+
if: repo =~ /^rust-lang\/rust-clippy$/ AND branch IN (auto, try)
6065
- env: INTEGRATION=rust-lang-nursery/log
61-
if: repo =~ /^rust-lang\/rust-clippy$/
66+
if: repo =~ /^rust-lang\/rust-clippy$/ AND branch IN (auto, try)
6267
- env: INTEGRATION=rust-lang-nursery/chalk
63-
if: repo =~ /^rust-lang\/rust-clippy$/
68+
if: repo =~ /^rust-lang\/rust-clippy$/ AND branch IN (auto, try)
6469
- env: INTEGRATION=rust-lang/rls
65-
if: repo =~ /^rust-lang\/rust-clippy$/
70+
if: repo =~ /^rust-lang\/rust-clippy$/ AND branch IN (auto, try)
6671
- env: INTEGRATION=chronotope/chrono
67-
if: repo =~ /^rust-lang\/rust-clippy$/
72+
if: repo =~ /^rust-lang\/rust-clippy$/ AND branch IN (auto, try)
6873
- env: INTEGRATION=serde-rs/serde
69-
if: repo =~ /^rust-lang\/rust-clippy$/
74+
if: repo =~ /^rust-lang\/rust-clippy$/ AND branch IN (auto, try)
7075
- env: INTEGRATION=Geal/nom
71-
if: repo =~ /^rust-lang\/rust-clippy$/
76+
if: repo =~ /^rust-lang\/rust-clippy$/ AND branch IN (auto, try)
7277
- env: INTEGRATION=hyperium/hyper
73-
if: repo =~ /^rust-lang\/rust-clippy$/
78+
if: repo =~ /^rust-lang\/rust-clippy$/ AND branch IN (auto, try)
7479
- env: INTEGRATION=bluss/rust-itertools
75-
if: repo =~ /^rust-lang\/rust-clippy$/
80+
if: repo =~ /^rust-lang\/rust-clippy$/ AND branch IN (auto, try)
7681
allow_failures:
7782
- os: windows
7883
env: CARGO_INCREMENTAL=0 BASE_TESTS=true

0 commit comments

Comments
 (0)