Skip to content

Commit b284165

Browse files
committed
Run all tests in ci
1 parent 290ea9b commit b284165

File tree

6 files changed

+14
-124
lines changed

6 files changed

+14
-124
lines changed

.github/scripts/overwrite_changeset_changelog.py

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

.github/workflows/code-qa.yml

Lines changed: 7 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -13,27 +13,6 @@ env:
1313
PNPM_VERSION: 10.8.1
1414

1515
jobs:
16-
compile:
17-
runs-on: ubuntu-latest
18-
steps:
19-
- name: Checkout code
20-
uses: actions/checkout@v4
21-
- name: Install pnpm
22-
uses: pnpm/action-setup@v4
23-
with:
24-
version: ${{ env.PNPM_VERSION }}
25-
- name: Setup Node.js
26-
uses: actions/setup-node@v4
27-
with:
28-
node-version: ${{ env.NODE_VERSION }}
29-
cache: 'pnpm'
30-
- name: Install dependencies
31-
run: pnpm install
32-
- name: Check types
33-
run: pnpm check-types
34-
- name: Lint
35-
run: pnpm lint
36-
3716
check-translations:
3817
runs-on: ubuntu-latest
3918
steps:
@@ -72,11 +51,8 @@ jobs:
7251
- name: Run knip checks
7352
run: pnpm knip
7453

75-
test-extension:
76-
runs-on: ${{ matrix.os }}
77-
strategy:
78-
matrix:
79-
os: [ubuntu-latest, windows-latest]
54+
compile:
55+
runs-on: ubuntu-latest
8056
steps:
8157
- name: Checkout code
8258
uses: actions/checkout@v4
@@ -91,11 +67,12 @@ jobs:
9167
cache: 'pnpm'
9268
- name: Install dependencies
9369
run: pnpm install
94-
- name: Run unit tests
95-
working-directory: src
96-
run: pnpm test
70+
- name: Lint
71+
run: pnpm lint
72+
- name: Check types
73+
run: pnpm check-types
9774

98-
test-webview:
75+
unit-test:
9976
runs-on: ${{ matrix.os }}
10077
strategy:
10178
matrix:
@@ -115,16 +92,8 @@ jobs:
11592
- name: Install dependencies
11693
run: pnpm install
11794
- name: Run unit tests
118-
working-directory: webview-ui
11995
run: pnpm test
12096

121-
unit-test:
122-
needs: [test-extension, test-webview]
123-
runs-on: ubuntu-latest
124-
steps:
125-
- name: NO-OP
126-
run: echo "All unit tests passed."
127-
12897
check-openrouter-api-key:
12998
runs-on: ubuntu-latest
13099
outputs:

packages/types/tsup.config.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ export default defineConfig({
44
entry: ["src/index.ts"],
55
format: ["cjs", "esm"],
66
dts: true,
7-
clean: true,
7+
clean: false,
88
splitting: false,
99
sourcemap: true,
1010
outDir: "dist",

src/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -320,7 +320,7 @@
320320
"scripts": {
321321
"lint": "eslint . --ext=ts --max-warnings=0",
322322
"check-types": "tsc --noEmit",
323-
"pretest": "pnpm --filter @roo-code/build build && pnpm --filter @roo-code/types build && node esbuild.mjs",
323+
"pretest": "turbo run bundle --cwd ..",
324324
"test": "jest -w=40% && vitest run --globals",
325325
"format": "prettier --write .",
326326
"bundle": "node esbuild.mjs",

turbo.json

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,10 @@
1010
"clean": {
1111
"cache": false
1212
},
13-
"build": {},
13+
"build": {
14+
"outputs": ["dist/**"],
15+
"inputs": ["src/**", "package.json", "tsconfig.json", "tsup.config.ts"]
16+
},
1417
"build:nightly": {},
1518
"bundle": {
1619
"dependsOn": ["@roo-code/types#build", "@roo-code/build#build", "@roo-code/vscode-webview#build:nightly"],

webview-ui/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
"scripts": {
66
"lint": "eslint src --ext=ts,tsx --max-warnings=0",
77
"check-types": "tsc",
8-
"pretest": "pnpm --filter @roo-code/types build",
8+
"pretest": "turbo run bundle --cwd ..",
99
"test": "jest -w=40%",
1010
"format": "prettier --write src",
1111
"dev": "vite",

0 commit comments

Comments
 (0)