Skip to content

Commit 623e775

Browse files
OttoAllmendingeraider
andcommitted
ci(root): run lint and format checks in parallel using matrix strategy
Ticket: BTC-1821 Co-authored-by: aider <[email protected]>
1 parent c259e2d commit 623e775

File tree

1 file changed

+38
-6
lines changed

1 file changed

+38
-6
lines changed

.github/workflows/ci.yml

Lines changed: 38 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -65,12 +65,6 @@ jobs:
6565
- name: Dependency Audit
6666
run: yarn run improved-yarn-audit --min-severity high
6767

68-
- name: Lint Source Code
69-
run: yarn run lint
70-
71-
- name: Check Source Code Formatting
72-
run: yarn run check-fmt
73-
7468
- name: build packages
7569
env:
7670
# Workaround for https://github.com/nodejs/node/issues/51555
@@ -93,6 +87,44 @@ jobs:
9387
# CODECOV_FLAG: unit
9488
# CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
9589

90+
code-quality:
91+
runs-on: ubuntu-latest
92+
strategy:
93+
fail-fast: false
94+
matrix:
95+
check: ['lint', 'format']
96+
97+
steps:
98+
- uses: actions/checkout@v4
99+
with:
100+
ref: ${{ github.event.pull_request.head.sha }}
101+
102+
- name: Setup node 18
103+
uses: actions/setup-node@v3
104+
with:
105+
node-version: 18
106+
107+
- name: restore lerna dependencies
108+
id: lerna-cache
109+
uses: actions/cache@v3
110+
with:
111+
path: |
112+
node_modules
113+
modules/*/node_modules
114+
key: ${{ runner.os }}-node18-${{ hashFiles('yarn.lock') }}-${{ hashFiles('tsconfig.packages.json') }}-${{ hashFiles('package.json') }}
115+
116+
- name: Install Packages
117+
if: steps.lerna-cache.outputs.cache-hit != 'true' || contains( github.event.pull_request.labels.*.name, 'SKIP_CACHE')
118+
run: yarn install --with-frozen-lockfile --ignore-scripts
119+
120+
- name: Lint Source Code
121+
if: matrix.check == 'lint'
122+
run: yarn run lint
123+
124+
- name: Check Source Code Formatting
125+
if: matrix.check == 'format'
126+
run: yarn run check-fmt
127+
96128
browser-test:
97129
runs-on: ubuntu-22.04
98130

0 commit comments

Comments
 (0)