Skip to content

Commit

Permalink
Switch to reusable ts ci workflow (#101)
Browse files Browse the repository at this point in the history
* Switch to reusable ts ci workflow

* Skip coverage

* Update to stable tag version
  • Loading branch information
skeet70 committed May 12, 2023
1 parent bb1a2df commit f13bc6c
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 44 deletions.
16 changes: 0 additions & 16 deletions .github/typescript-ci-patch.yaml

This file was deleted.

35 changes: 9 additions & 26 deletions .github/workflows/typescript-ci.yaml
Original file line number Diff line number Diff line change
@@ -1,33 +1,16 @@
# DO NOT EDIT THIS FILE.
# Instead, edit the jsonpatch file (actually YAML) in .github/typescript-ci-patch.yaml
# For docs, see github-actions in the IronCoreLabs/depot repo.

name: TypeScript CI
concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: true
name: Typescript CI
'on':
push:
branches:
- main
pull_request: null
workflow_dispatch: null
jobs:
test:
runs-on: ubuntu-22.04
strategy:
matrix:
node_version:
- '14'
- '16'
- '18'
steps:
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
with:
node-version: ${{ matrix.node_version }}
cache: yarn
- name: Install modules
run: yarn
- name: Run tests
run: yarn test --coverage
typescript-ci:
uses: IronCoreLabs/workflows/.github/workflows/typescript-ci.yaml@typescript-ci-v0
with:
test_matrix_node_version: '["14", "16", "18"]'
# this repo runs coverage in its default test command, which will fail if
# under the thresholds defined in jest.config.js, so coverage delta doesn't matter as much
run_coverage: false
secrets: inherit
3 changes: 2 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,5 @@ dist/
coverage/*
integration/private.key
integration/.device.json
integration/project.json
integration/project.json
.direnv
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
"start": "_ICL_API_DOMAIN_REPLACEMENT_=http://localhost:9090 ts-node -- ./integration/index.ts",
"build": "./build.js",
"test": "yarn run lint && yarn run unit",
"unit": "jest",
"unit": "jest --coverage",
"lint": "tslint -p \"tsconfig.json\" -e \"**/tests/**\" \"src/**/*.ts\""
},
"dependencies": {
Expand Down

0 comments on commit f13bc6c

Please sign in to comment.