Skip to content

Commit df45d06

Browse files
committed
ci: include optional deps in npm ci
1 parent b29de66 commit df45d06

File tree

2 files changed

+3
-8
lines changed

2 files changed

+3
-8
lines changed

.github/workflows/cli-pr-checks.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ jobs:
3636
- name: Install dependencies
3737
run: |
3838
cd extensions/cli
39-
npm ci
39+
npm ci --include=optional
4040
4141
- name: Run linting
4242
run: |
@@ -72,7 +72,7 @@ jobs:
7272
- name: Install dependencies
7373
run: |
7474
cd extensions/cli
75-
npm ci
75+
npm ci --include=optional
7676
7777
- name: Build
7878
run: |

extensions/cli/build.mjs

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,12 @@
11
#!/usr/bin/env node
22

33
import * as esbuild from "esbuild";
4-
import { readFileSync, writeFileSync, chmodSync } from "fs";
4+
import { writeFileSync, chmodSync } from "fs";
55
import { resolve, dirname } from "path";
66
import { fileURLToPath } from "url";
77

88
const __dirname = dirname(fileURLToPath(import.meta.url));
99

10-
// Read package.json to get dependencies
11-
const packageJson = JSON.parse(
12-
readFileSync(resolve(__dirname, "package.json"), "utf8"),
13-
);
14-
1510
// List of packages to mark as external (ONLY native modules that cannot be bundled)
1611
// Note: Everything else will be bundled to create a self-contained CLI
1712
// Users should not need to install any additional dependencies

0 commit comments

Comments
 (0)