Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use consistent case for AS in Dockerfile #2

Merged
merged 1 commit into from
Jul 23, 2024
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
FROM node:hydrogen-alpine as base
FROM node:hydrogen-alpine AS base

WORKDIR /usr/src/app

Expand All @@ -9,7 +9,7 @@ RUN deluser --remove-home node \
&& addgroup -S node -g 10000 \
&& adduser -S -G node -u 10000 node

FROM base as build
FROM base AS build

COPY package-lock.json ./
COPY package.json ./
Expand All @@ -20,7 +20,7 @@ RUN chmod u+x ./server.js
RUN npm pack
RUN tar -xzf *.tgz

FROM base as install
FROM base AS install

ENV NODE_ENV=production

Expand All @@ -29,7 +29,7 @@ COPY package.json ./
RUN --mount=type=cache,target=/root/.npm npm ci
RUN rm package.json package-lock.json

FROM base as app
FROM base AS app

COPY --from=install /usr/src/app .
COPY --from=build /usr/src/app/package .
Expand Down
Loading