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

Don't run Electron tests on Travis #947

Closed
wants to merge 1 commit into from
Closed
Changes from all commits
Commits
File filter...
Filter file types
Jump to…
Jump to file or symbol
Failed to load files and symbols.

Always

Just for now

Don't run Electron tests on Travis

This is a workaround for #945

Note that pull requests from other repos will still run the Electron
tests (and remind us that it's broken), but at least master will be
green again.
  • Loading branch information
josephfrazier committed Oct 12, 2016
commit 9f85815fb36a6b582b043ec405ba07f6772eff11
@@ -6,11 +6,11 @@ var runSauceLabs = !process.env.CI ||
(process.env.SAUCE_USERNAME && process.env.SAUCE_ACCESS_KEY)

npmRun('test-node', function () {
npmRun('test-browser-headless', function () {
if (runSauceLabs) {
npmRun('test-browser')
}
})
if (runSauceLabs) {
npmRun('test-browser')
} else {
npmRun('test-browser-headless')
}
})

function npmRun (scriptName, onSuccess) {
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.