👷 (pr): Update PR CI workflow to include typechecks

LedgerHQ · Feb 14, 2024 · 0717aea · 0717aea
1 parent 595958e
commit 0717aea
Show file tree

Hide file tree

Showing 8 changed files with 17 additions and 25 deletions.
diff --git a/.github/workflows/pull_request.yml b/.github/workflows/pull_request.yml
@@ -1,42 +1,25 @@
 name: pull_request
 on: [pull_request]
+
+env:
+  FORCE_COLOR: "1"
+
 jobs:
-  lint:
+  health-check:
     runs-on: [device-sdk-4xlarge-linux]
-    env:
-      CI_JOB_NUMBER: 1
     steps:
       - uses: actions/checkout@v3
 
       - uses: ./actions/setup-toolchain-composite
 
       - name: Lint
-        env:
-          FORCE_COLOR: "1"
         run: pnpm lint
 
-  prettier:
-    runs-on: [device-sdk-4xlarge-linux]
-    env:
-      CI_JOB_NUMBER: 1
-    steps:
-      - uses: actions/checkout@v3
-
-      - uses: ./actions/setup-toolchain-composite
-
       - name: Prettier
         run: pnpm prettier
 
-  test:
-    runs-on: [device-sdk-4xlarge-linux]
-    env:
-      CI_JOB_NUMBER: 1
-    steps:
-      - uses: actions/checkout@v3
-
-      - uses: ./actions/setup-toolchain-composite
+      - name: Typecheck
+        run: pnpm typecheck
 
       - name: Test
-        env:
-          FORCE_COLOR: "1"
         run: pnpm test
diff --git a/apps/sample/package.json b/apps/sample/package.json
@@ -9,7 +9,8 @@
     "lint": "next lint --max-warnings=0",
     "lint:fix": "next lint --fix",
     "prettier": "prettier . --check",
-    "prettier:fix": "prettier . --write"
+    "prettier:fix": "prettier . --write",
+    "typecheck": "tsc --noEmit"
   },
   "dependencies": {
     "@ledgerhq/device-sdk-core": "workspace:*",

diff --git a/package.json b/package.json
@@ -15,6 +15,7 @@
     "prettier": "turbo run prettier --log-order=grouped",
     "prettier:fix": "turbo run prettier:fix --log-order=grouped",
     "test": "turbo run test --log-order=grouped",
+    "typecheck": "turbo run typecheck --log-order=grouped",
     "core": "pnpm --filter @ledgerhq/device-sdk-core",
     "signer": "pnpm --filter @ledgerhq/device-sdk-signer",
     "trusted-apps": "pnpm --filter @ledgerhq/device-sdk-trusted-apps",

diff --git a/packages/core/package.json b/packages/core/package.json
@@ -17,6 +17,7 @@
     "test": "jest src",
     "test:watch": "pnpm test -- --watch",
     "test:coverage": "pnpm test -- --coverage",
+    "typecheck": "tsc --noEmit",
     "module:create": "pnpm hygen core-module with-prompt"
   },
   "dependencies": {

diff --git a/packages/signer/package.json b/packages/signer/package.json
@@ -14,6 +14,7 @@
     "lint:fix": "eslint --cache --fix --ext .ts \"src\"",
     "prettier": "prettier . --check",
     "prettier:fix": "prettier . --write",
+    "typecheck": "tsc --noEmit",
     "test": "jest --passWithNoTests"
   },
   "dependencies": {

diff --git a/packages/trusted-apps/package.json b/packages/trusted-apps/package.json
@@ -13,6 +13,7 @@
     "lint:fix": "eslint --cache --fix --ext .ts \"src\"",
     "prettier": "prettier . --check",
     "prettier:fix": "prettier . --write",
+    "typecheck": "tsc --noEmit",
     "dev": "tsc --watch",
     "test": "jest --passWithNoTests"
   },

diff --git a/packages/ui/package.json b/packages/ui/package.json
@@ -14,6 +14,7 @@
     "lint:fix": "eslint --cache --fix --ext .ts \"src\"",
     "prettier": "prettier . --check",
     "prettier:fix": "prettier . --write",
+    "typecheck": "tsc --noEmit",
     "test": "jest --passWithNoTests"
   },
   "dependencies": {

diff --git a/turbo.json b/turbo.json
@@ -27,6 +27,9 @@
       "dependsOn": ["build", "lint"],
       "outputs": [],
       "inputs": ["src/**/*.ts", "tests/**/*.ts"]
+    },
+    "typecheck": {
+      "dependsOn": ["^typecheck"]
     }
   }
 }