Skip to content

Commit 0665921

Browse files
committed
fix(ci): resolve build and test pipeline issues
- Add build dependency to type-check, test:coverage, and test:e2e tasks - Use turbo for test:e2e to ensure proper dependency ordering - Remove semantic-release git push plugins (branch protection compat) - Make drizzle.config.ts resilient to missing DATABASE_URL for knip - Correct node engine version to >=22.0.0
1 parent fd5e861 commit 0665921

7 files changed

Lines changed: 22 additions & 3084 deletions

File tree

.github/workflows/ci.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,6 @@ jobs:
103103

104104
- name: Run E2E tests
105105
run: pnpm test:e2e
106-
working-directory: apps/nextjs
107106
env:
108107
CI: true
109108

knip.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,8 @@
1717
"ignore": ["**/*.test.ts", "**/*.spec.ts", "**/stubs/**"]
1818
},
1919
"packages/drizzle": {
20-
"project": ["src/**/*.ts"]
20+
"project": ["src/**/*.ts"],
21+
"ignore": ["drizzle.config.ts"]
2122
},
2223
"packages/ui": {
2324
"entry": ["src/**/*.tsx", "src/**/*.ts"],

package.json

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
"test:watch": "turbo test:watch",
2929
"test:coverage": "turbo test:coverage",
3030
"test:ui": "vitest --ui",
31-
"test:e2e": "pnpm --filter=nextjs test:e2e",
31+
"test:e2e": "turbo test:e2e",
3232
"setup": "pnpm install && pnpm db && pnpm db:push",
3333
"validate": "pnpm type-check && pnpm check && pnpm test",
3434
"quick-check": "pnpm type-check && pnpm check",
@@ -38,8 +38,6 @@
3838
"@biomejs/biome": "^2.3.10",
3939
"@commitlint/cli": "^20.3.1",
4040
"@commitlint/config-conventional": "^20.3.1",
41-
"@semantic-release/changelog": "^6.0.3",
42-
"@semantic-release/git": "^10.0.1",
4341
"husky": "^9.1.7",
4442
"jscpd": "^4.0.7",
4543
"knip": "^5.81.0",
@@ -57,6 +55,6 @@
5755
},
5856
"packageManager": "pnpm@10.26.2",
5957
"engines": {
60-
"node": ">=24.12.0"
58+
"node": ">=22.0.0"
6159
}
6260
}

packages/drizzle/drizzle.config.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,14 +4,14 @@ import { defineConfig } from "drizzle-kit";
44

55
config({ path: resolve(__dirname, "../../.env") });
66

7-
if (!process.env.DATABASE_URL)
8-
throw new Error("DATABASE_URL is not defined in root .env file");
7+
// Use placeholder for static analysis tools (knip), actual value required at runtime
8+
const databaseUrl = process.env.DATABASE_URL ?? "postgresql://placeholder";
99

1010
export default defineConfig({
1111
schema: "./src/schema/*",
1212
out: "./migrations",
1313
dbCredentials: {
14-
url: process.env.DATABASE_URL,
14+
url: databaseUrl,
1515
},
1616
verbose: true,
1717
strict: true,

0 commit comments

Comments
 (0)