Skip to content

Commit 62da7c0

Browse files
author
Akash Satheesan
authored
refactor(ci): split codecov/audit from prebuild
Move codecov calculations and dependency audits from prebuild to their own jobs, so that an error does not affect the rest of the build/test process.
1 parent af5a1c9 commit 62da7c0

File tree

1 file changed

+50
-4
lines changed

1 file changed

+50
-4
lines changed

.github/workflows/ci.yaml

+50-4
Original file line numberDiff line numberDiff line change
@@ -43,10 +43,6 @@ jobs:
4343
if: steps.cache-yarn.outputs.cache-hit != 'true'
4444
run: yarn --frozen-lockfile
4545

46-
- name: Audit for vulnerabilities
47-
run: yarn _audit
48-
if: success()
49-
5046
- name: Run yarn fmt
5147
run: yarn fmt
5248
if: success()
@@ -59,10 +55,60 @@ jobs:
5955
run: yarn test:unit
6056
if: success()
6157

58+
codecov:
59+
name: Upload coverage report to Codecov
60+
runs-on: ubuntu-latest
61+
steps:
62+
- name: Checkout repo
63+
uses: actions/checkout@v2
64+
65+
- name: Install Node.js v12
66+
uses: actions/setup-node@v2
67+
with:
68+
node-version: "12"
69+
70+
- name: Fetch dependencies from cache
71+
id: cache-yarn
72+
uses: actions/cache@v2
73+
with:
74+
path: "**/node_modules"
75+
key: yarn-build-${{ hashFiles('**/yarn.lock') }}
76+
77+
- name: Install dependencies
78+
if: steps.cache-yarn.outputs.cache-hit != 'true'
79+
run: yarn --frozen-lockfile
80+
6281
- name: Upload coverage report to Codecov
6382
run: yarn coverage
6483
if: success()
6584

85+
audit-ci:
86+
name: Run audit-ci
87+
runs-on: ubuntu-latest
88+
steps:
89+
- name: Checkout repo
90+
uses: actions/checkout@v2
91+
92+
- name: Install Node.js v12
93+
uses: actions/setup-node@v2
94+
with:
95+
node-version: "12"
96+
97+
- name: Fetch dependencies from cache
98+
id: cache-yarn
99+
uses: actions/cache@v2
100+
with:
101+
path: "**/node_modules"
102+
key: yarn-build-${{ hashFiles('**/yarn.lock') }}
103+
104+
- name: Install dependencies
105+
if: steps.cache-yarn.outputs.cache-hit != 'true'
106+
run: yarn --frozen-lockfile
107+
108+
- name: Audit for vulnerabilities
109+
run: yarn _audit
110+
if: success()
111+
66112
build:
67113
name: Build
68114
needs: prebuild

0 commit comments

Comments
 (0)