Skip to content

Commit 9f7d922

Browse files
authored
build(ci): Remove TravisCI (#3149)
Travis sprung some surprise billing changes on us, so we are now prioritizing removing TravisCI completely.
1 parent beaf207 commit 9f7d922

File tree

5 files changed

+21
-44
lines changed

5 files changed

+21
-44
lines changed

.github/workflows/build.yml

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ jobs:
1010
job_build:
1111
name: Build
1212
runs-on: ubuntu-latest
13+
timeout-minutes: 15
1314
steps:
1415
- uses: actions/checkout@v2
1516
- uses: actions/setup-node@v1
@@ -31,6 +32,7 @@ jobs:
3132
job_size_check:
3233
name: Size Check
3334
needs: job_build
35+
timeout-minutes: 15
3436
runs-on: ubuntu-latest
3537
if: ${{ github.head_ref }}
3638
steps:
@@ -54,6 +56,7 @@ jobs:
5456
job_lint:
5557
name: Lint
5658
needs: job_build
59+
timeout-minutes: 10
5760
runs-on: ubuntu-latest
5861
steps:
5962
- uses: actions/checkout@v2
@@ -75,10 +78,17 @@ jobs:
7578
job_unit_test:
7679
name: Test
7780
needs: job_build
81+
continue-on-error: true
82+
timeout-minutes: 30
7883
runs-on: ubuntu-latest
84+
strategy:
85+
matrix:
86+
node: [6, 8, 10, 12, 14]
7987
steps:
8088
- uses: actions/checkout@v2
8189
- uses: actions/setup-node@v1
90+
with:
91+
node-version: ${{ matrix.node }}
8292
- uses: actions/cache@v2
8393
with:
8494
path: |
@@ -89,9 +99,10 @@ jobs:
8999
${{ github.workspace }}/packages/**/esm
90100
${{ github.workspace }}/packages/serverless/dist-awslambda-layer/*.zip
91101
key: ${{ github.sha }}
92-
- run: yarn install
93102
- name: Unit Tests
94-
run: yarn test
103+
env:
104+
NODE_VERSION: ${{ matrix.node }}
105+
run: ./scripts/test.sh
95106
- uses: codecov/codecov-action@v1
96107

97108
job_artifacts:
@@ -129,6 +140,7 @@ jobs:
129140
name: BrowserStack
130141
needs: job_build
131142
runs-on: ubuntu-latest
143+
timeout-minutes: 45
132144
if: "github.ref == 'refs/heads/master'"
133145
steps:
134146
- uses: actions/checkout@v2

.travis.yml

Lines changed: 0 additions & 37 deletions
This file was deleted.

packages/browser/test/integration/browsers.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ module.exports = {
2323
bs_android_6: {
2424
base: "BrowserStack",
2525
browser: "Android Browser",
26-
device: "Samsung Galaxy Note 4",
26+
device: "Google Nexus 6",
2727
os: "android",
2828
os_version: "6.0",
2929
real_mobile: true,

packages/browser/test/integration/karma.conf.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,8 @@ const browsers = isLocalRun ? ["ChromeHeadless"] : Object.keys(customLaunchers);
1212
// for each browser here, so that we have a nice distinction of when the tests were run exactly.
1313
if (!isLocalRun) {
1414
for (const browser in customLaunchers) {
15-
customLaunchers[browser].build = process.env.TRAVIS_BUILD_NUMBER
16-
? `Travis: ${process.env.TRAVIS_BUILD_NUMBER}`
15+
customLaunchers[browser].build = process.env.GITHUB_RUN_ID
16+
? `CI: ${process.env.GITHUB_RUN_ID}`
1717
: `Manual: ${new Date().toLocaleString()}`;
1818
}
1919
}

scripts/test.sh

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,8 @@ set -e
33
source ~/.nvm/nvm.sh
44

55
# We need this check to skip engines check for typescript-tslint-plugin package
6-
if [[ "$(cut -d. -f1 <<< "$TRAVIS_NODE_VERSION")" -le 6 ]]; then
6+
if [[ "$(cut -d. -f1 <<< "$NODE_VERSION")" -le 6 ]]; then
7+
nvm install 8
78
nvm use 8
89
yarn install --ignore-engines --ignore-scripts
910
# current versions of nock don't support node 6
@@ -12,10 +13,11 @@ if [[ "$(cut -d. -f1 <<< "$TRAVIS_NODE_VERSION")" -le 6 ]]; then
1213
cd ../..
1314
# ember requires Node >= 10 to build
1415
yarn build --ignore="@sentry/ember" --ignore="@sentry/serverless" --ignore="@sentry/gatsby" --ignore="@sentry/react"
16+
nvm install 6
1517
nvm use 6
1618
# browser can be tested only on Node >= v8 because Karma is not supporting anything older
1719
yarn test --ignore="@sentry/tracing" --ignore="@sentry/react" --ignore="@sentry/gatsby" --ignore="@sentry/ember" --ignore="@sentry-internal/eslint-plugin-sdk" --ignore="@sentry-internal/eslint-config-sdk" --ignore="@sentry/serverless" --ignore="@sentry/browser" --ignore="@sentry/integrations"
18-
elif [[ "$(cut -d. -f1 <<< "$TRAVIS_NODE_VERSION")" -le 8 ]]; then
20+
elif [[ "$(cut -d. -f1 <<< "$NODE_VERSION")" -le 8 ]]; then
1921
yarn install --ignore-engines --ignore-scripts
2022
# ember requires Node >= 10 to build
2123
yarn build --ignore="@sentry/ember" --ignore="@sentry/serverless" --ignore="@sentry/gatsby" --ignore="@sentry/react"

0 commit comments

Comments
 (0)