Skip to content

Commit 39c44ec

Browse files
committed
Update PR #2759
1 parent 0bd5da1 commit 39c44ec

2 files changed

Lines changed: 5 additions & 12 deletions

File tree

‎integrations/otel-js/Dockerfile.test‎

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -33,17 +33,10 @@ WORKDIR /workspace
3333
# Copy the entire SDK repository (build context is SDK repo root)
3434
COPY . .
3535

36-
# Build braintrust SDK first (required peer dependency)
37-
WORKDIR /workspace/js
38-
RUN pnpm install --ignore-workspace
39-
RUN pnpm run build
40-
41-
# Build @braintrust/otel package
42-
WORKDIR /workspace/integrations/otel-js
43-
# Replace workspace:* with file: path for braintrust dependency
44-
RUN sed -i.bak 's/"braintrust": "workspace:\*"/"braintrust": "file:..\/..\/js"/' package.json && rm -f package.json.bak
45-
RUN pnpm install --ignore-workspace
46-
RUN pnpm run build
36+
# Build both packages with the workspace lockfile so peer dependencies do not
37+
# resolve to untested releases. Tests install their own version-specific deps.
38+
RUN pnpm install --frozen-lockfile
39+
RUN pnpm exec turbo run build --filter=braintrust --filter=@braintrust/otel
4740

4841
# Set the test directory as environment variable (available at runtime)
4942
ENV TEST_DIR=${TEST_DIR}

‎integrations/otel-js/run-tests.sh‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
#!/bin/sh
22
set -e
33

4-
pnpm install --ignore-workspace
4+
pnpm install --ignore-workspace --frozen-lockfile
55
pnpm exec vitest run

0 commit comments

Comments
 (0)