Skip to content

Commit 9bbfd7c

Browse files
fix(docker): update Dockerfile.temporal-worker-main for build process
- Added a comment to disable SonarCloud warning for Dockerfile best practices. - Removed the unnecessary `ls -al` command from the build step to streamline the Dockerfile. These modifications enhance the clarity and efficiency of the Docker build process for the Temporal worker.
1 parent 6edcf5a commit 9bbfd7c

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

Dockerfile.temporal-worker-main

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ RUN npm ci
66
FROM node:20-bullseye AS dev
77
WORKDIR /app
88
COPY --from=deps /app/node_modules ./node_modules
9+
# sonarcloud-disable-next-line docker:S4507
910
ENV NODE_ENV=development
1011
COPY workers/main ./
1112
CMD ["npx", "nodemon", "--watch", "./", "--ext", "ts", "--exec", "npx", "ts-node", "src/index.ts"]
@@ -14,7 +15,7 @@ FROM node:20-bullseye AS build
1415
WORKDIR /app
1516
COPY --from=deps /app/node_modules ./node_modules
1617
COPY workers/main/ ./
17-
RUN npm run build && ls -al
18+
RUN npm run build
1819

1920
FROM gcr.io/distroless/nodejs20-debian11 AS production
2021
WORKDIR /app

0 commit comments

Comments
 (0)