chore: disable optional travis modes #2549

Merged
merged 2 commits into from Jan 6, 2017

Projects

None yet

4 participants

@DevVersion
Member
  • Temporary disable optional travis modes because those always fail at the moment and also can be potential culprits for CI flakiness on Saucelabs & Browserstack. See #2523
@DevVersion DevVersion chore: disable optional travis modes
* Temporary disable optional travis modes because those always fail at the moment and also can be potential culprits for CI flakiness on Saucelabs & Browserstack
d475556
@DevVersion DevVersion requested a review from jelbourn Jan 6, 2017
@googlebot googlebot added the cla: yes label Jan 6, 2017
.travis.yml
- - MODE=browserstack_optional
+# Temporary disable the optional builds because of potential concurrency flakiness
+# - MODE=saucelabs_optional
+# - MODE=browserstack_optional
@jelbourn
jelbourn Jan 6, 2017 Member

Rather than commenting it out, just remove them completely. Maybe someday we'll add them back.

@DevVersion
DevVersion Jan 6, 2017 Member

Removed everything now.

@DevVersion DevVersion Remove optional builds completely
bf52f66
@jelbourn

LGTM

@kara kara merged commit 0be5acd into angular:master Jan 6, 2017

2 checks passed

cla/google All necessary CLAs are signed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment