Skip to content

Commit 8413af5

Browse files
committed
fix: use path.delimiter instead of hard-coded colon in NODE_PATH join
1 parent 121cda1 commit 8413af5

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

  • internal-packages/testcontainers/src

internal-packages/testcontainers/src/webapp.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { spawn } from "child_process";
22
import { createServer } from "net";
3-
import { resolve } from "path";
3+
import { delimiter, resolve } from "path";
44
import { Network } from "testcontainers";
55
import { PrismaClient } from "@trigger.dev/database";
66
import { createPostgresContainer } from "./utils";
@@ -46,7 +46,7 @@ export async function startWebapp(databaseUrl: string): Promise<{
4646
// Merge NODE_PATH so transitive pnpm deps (hoisted to .pnpm/node_modules) are resolvable
4747
const existingNodePath = process.env.NODE_PATH;
4848
const nodePath = existingNodePath
49-
? `${PNPM_HOISTED_MODULES}:${existingNodePath}`
49+
? `${PNPM_HOISTED_MODULES}${delimiter}${existingNodePath}`
5050
: PNPM_HOISTED_MODULES;
5151

5252
const proc = spawn(process.execPath, ["build/server.js"], {

0 commit comments

Comments
 (0)