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

chore: run windows build on Travis CI #7211

Closed
wants to merge 13 commits into from
19 changes: 17 additions & 2 deletions .travis.yml
@@ -1,13 +1,28 @@
language: node_js

node_js:
- '10'
node_js: 10

env:
- YARN_GPG=no

sudo: false

os:
- windows
- linux


before_install:
- curl -o- -L https://yarnpkg.com/install.sh | bash
- export PATH="$HOME/.yarn/bin:$PATH"
- if [[ $TRAVIS_OS_NAME == windows ]]; then
choco install python2;
export PATH="/c/Python27:/c/Python27/Scripts:$PATH";
fi

branches:
only:
- master
thymikee marked this conversation as resolved.
Show resolved Hide resolved

cache:
yarn: true
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Expand Up @@ -110,6 +110,7 @@
- `[jest-worker]` Standardize filenames ([#7316](https://github.com/facebook/jest/pull/7316))
- `[pretty-format]` Standardize filenames ([#7316](https://github.com/facebook/jest/pull/7316))
- `[*]` Add check for Facebook copyright headers on CI ([#7370](https://github.com/facebook/jest/pull/7370))
- `[ci]` Run Windows builds on Travis ([#7211](https://github.com/facebook/jest/pull/7211))

### Performance

Expand Down
1 change: 0 additions & 1 deletion README.md
Expand Up @@ -7,7 +7,6 @@
<p align="center">
<a href="https://circleci.com/gh/facebook/jest"><img src="https://circleci.com/gh/facebook/jest.svg?style=shield" alt="CircleCI Build Status"></a>
<a href="https://travis-ci.org/facebook/jest"><img src="https://travis-ci.org/facebook/jest.svg?branch=master" alt="Travis Build Status"></a>
<a href="https://ci.appveyor.com/project/Daniel15/jest/branch/master"><img src="https://ci.appveyor.com/api/projects/status/8n38o44k585hhvhd/branch/master?svg=true" alt="Windows Build Status"></a>
<a href="https://codecov.io/gh/facebook/jest"><img src="https://codecov.io/gh/facebook/jest/branch/master/graph/badge.svg" alt="Codecov badge"></a>
<a href="http://badge.fury.io/js/jest"><img src="https://badge.fury.io/js/jest.svg" alt="npm version"></a>
<a href="https://dependabot.com/compatibility-score.html?dependency-name=jest&amp;package-manager=npm_and_yarn&amp;version-scheme=semver"><img src="https://api.dependabot.com/badges/compatibility_score?dependency-name=jest&amp;package-manager=npm_and_yarn&amp;version-scheme=semver" alt="SemVer">
Expand Down
31 changes: 0 additions & 31 deletions appveyor.yml

This file was deleted.

1 change: 0 additions & 1 deletion package.json
Expand Up @@ -43,7 +43,6 @@
"isbinaryfile": "^3.0.3",
"istanbul-api": "^1.3.1",
"istanbul-lib-coverage": "^1.0.0",
"jasmine-reporters": "^2.2.0",
"jest-junit": "^5.1.0",
"jest-silent-reporter": "^0.1.1",
"jquery": "^3.2.1",
Expand Down
7 changes: 0 additions & 7 deletions testSetupFile.js
Expand Up @@ -5,13 +5,6 @@
* LICENSE file in the root directory of this source tree.
*/

const jasmineReporters = require('jasmine-reporters');

// Some of the `jest-runtime` tests are very slow and cause
// timeouts on travis
jest.setTimeout(70000);

if (global.jasmine && process.env.APPVEYOR_API_URL) {
// Running on AppVeyor, add the custom reporter.
jasmine.getEnv().addReporter(new jasmineReporters.AppVeyorReporter());
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Too bad Travis doesn't have any concept of test result reporters. This is actually very useful as it pulls out the test results into a separate table view rather than having to look through pages of console output.

image

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, would be nice if they at least support junit reports...

}
10 changes: 1 addition & 9 deletions yarn.lock
Expand Up @@ -7936,14 +7936,6 @@ iterall@^1.2.2:
resolved "https://registry.yarnpkg.com/iterall/-/iterall-1.2.2.tgz#92d70deb8028e0c39ff3164fdbf4d8b088130cd7"
integrity sha512-yynBb1g+RFUPY64fTrFv7nsjRrENBQJaX2UL+2Szc9REFrSNm1rpSXHGzhmAy7a9uv3vlvgBlXnf9RqmPH1/DA==

jasmine-reporters@^2.2.0:
version "2.3.2"
resolved "https://registry.yarnpkg.com/jasmine-reporters/-/jasmine-reporters-2.3.2.tgz#898818ffc234eb8b3f635d693de4586f95548d43"
integrity sha512-u/7AT9SkuZsUfFBLLzbErohTGNsEUCKaQbsVYnLFW1gEuL2DzmBL4n8v90uZsqIqlWvWUgian8J6yOt5Fyk/+A==
dependencies:
mkdirp "^0.5.1"
xmldom "^0.1.22"

jest-docblock@^21.0.0:
version "21.2.0"
resolved "https://registry.yarnpkg.com/jest-docblock/-/jest-docblock-21.2.0.tgz#51529c3b30d5fd159da60c27ceedc195faf8d414"
Expand Down Expand Up @@ -14142,7 +14134,7 @@ xmldoc@^0.4.0:
dependencies:
sax "~1.1.1"

xmldom@0.1.x, xmldom@^0.1.22:
xmldom@0.1.x:
version "0.1.27"
resolved "https://registry.yarnpkg.com/xmldom/-/xmldom-0.1.27.tgz#d501f97b3bdb403af8ef9ecc20573187aadac0e9"
integrity sha1-1QH5ezvbQDr4757MIFcxh6rawOk=
Expand Down