Skip to content

Commit bba72c1

Browse files
authored
chore: run tsc and linting as GH action (#9223)
1 parent 02e8b31 commit bba72c1

3 files changed

Lines changed: 48 additions & 18 deletions

File tree

.circleci/config.yml

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -18,19 +18,6 @@ aliases:
1818

1919
version: 2
2020
jobs:
21-
lint-and-typecheck:
22-
working_directory: ~/jest
23-
docker:
24-
- image: circleci/node:12
25-
steps:
26-
- checkout
27-
- restore-cache: *restore-cache
28-
- run: yarn --no-progress --frozen-lockfile
29-
- save-cache: *save-cache
30-
- run: yarn lint --format junit -o reports/junit/js-lint-results.xml && yarn lint-es5-build --format junit -o reports/junit/js-es5-lint-results.xml && yarn lint:prettier:ci && yarn check-copyright-headers
31-
- store_test_results:
32-
path: reports/junit
33-
3421
test-node-8:
3522
working_directory: ~/jest
3623
docker:
@@ -131,7 +118,6 @@ workflows:
131118
version: 2
132119
build-and-deploy:
133120
jobs:
134-
- lint-and-typecheck
135121
- test-node-8
136122
- test-node-10
137123
- test-node-12

.github/workflows/nodejs.yml

Lines changed: 43 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,43 @@
1+
name: Node CI
2+
3+
on:
4+
push:
5+
branches:
6+
- master
7+
pull_request:
8+
branches:
9+
- '**'
10+
11+
jobs:
12+
lint-and-typecheck:
13+
name: Running TypeScript compiler & ESLint
14+
runs-on: ubuntu-latest
15+
16+
steps:
17+
- uses: actions/checkout@v1
18+
- name: Get yarn cache
19+
id: yarn-cache
20+
run: echo "::set-output name=dir::$(yarn cache dir)"
21+
- uses: actions/cache@v1
22+
with:
23+
path: ${{ steps.yarn-cache.outputs.dir }}
24+
key: ubuntu-latest-node-12.x-yarn-${{ hashFiles('**/yarn.lock') }}
25+
restore-keys: |
26+
ubuntu-latest-node-12.x-yarn-
27+
- uses: actions/setup-node@v1
28+
with:
29+
node-version: 12.x
30+
- name: install
31+
run: yarn install-no-ts-build
32+
- name: build
33+
run: node scripts/build.js
34+
- name: run tsc
35+
run: yarn build:ts
36+
- name: run eslint
37+
run: yarn lint
38+
- name: run eslint on browser builds
39+
run: yarn lint-es5-build
40+
- name: run prettier
41+
run: yarn lint:prettier:ci
42+
- name: check copyright headers
43+
run: yarn check-copyright-headers

scripts/buildTs.js

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ const path = require('path');
1212

1313
const chalk = require('chalk');
1414
const execa = require('execa');
15-
const {getPackages, adjustToTerminalWidth, OK} = require('./buildUtils');
15+
const {getPackages} = require('./buildUtils');
1616

1717
const packages = getPackages();
1818

@@ -33,13 +33,14 @@ const args = [
3333
];
3434

3535
console.log(chalk.inverse('Building TypeScript definition files'));
36-
process.stdout.write(adjustToTerminalWidth('Building\n'));
3736

3837
try {
3938
execa.sync('node', args, {stdio: 'inherit'});
40-
process.stdout.write(`${OK}\n`);
39+
console.log('Successfully built TypeScript definition files');
40+
console.log(
41+
chalk.inverse.green('Successfully built TypeScript definition files')
42+
);
4143
} catch (e) {
42-
process.stdout.write('\n');
4344
console.error(
4445
chalk.inverse.red('Unable to build TypeScript definition files')
4546
);

0 commit comments

Comments
 (0)