Skip to content

Commit

Permalink
Auto merge of #4028 - phansch:skip_appveyor_on_pr_pushes, r=phansch
Browse files Browse the repository at this point in the history
Only run AppVeyor on r+, try and the master branch

As it is right now, there is only one worker available in the `rust-lang-libs`
AppVeyor project and there are other repos as well that we share this worker
with. This has been a problem for us because we sometimes hit a bors timeout if there
are too many builds queued up.

To improve the situation, I think we could try to use AppVeyor a bit less
often. The average PR is not going to break windows related things anyway, so
it should be fine to run it on r+/try/master only.

changelog: none
  • Loading branch information
bors committed Apr 25, 2019
2 parents d84d3f8 + 8d7a5fe commit dbaa6f0
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions appveyor.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,11 +8,11 @@ environment:
- TARGET: x86_64-pc-windows-msvc

branches:
# Don't build these branches
except:
# Used by bors
- trying.tmp
- staging.tmp
# Only build AppVeyor on r+, try and the master branch
only:
- auto
- try
- master

install:
- curl -sSf -o rustup-init.exe https://win.rustup.rs/
Expand Down

0 comments on commit dbaa6f0

Please sign in to comment.