Skip to content

[서석규] sprint11#159

Open
ssk7594-CRz wants to merge 8 commits into
codeit-bootcamp-nodejs:서석규from
ssk7594-CRz:feat/docker-github-actions
Open

[서석규] sprint11#159
ssk7594-CRz wants to merge 8 commits into
codeit-bootcamp-nodejs:서석규from
ssk7594-CRz:feat/docker-github-actions

Conversation

@ssk7594-CRz
Copy link
Copy Markdown
Collaborator

요구사항

기본

  • 기본 항목 1
  • 기본 항목 2

심화

  • 심화 항목 1
  • 심화 항목 2

주요 변경사항

스크린샷

image

멘토에게

  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

Copy link
Copy Markdown
Collaborator

@ac9831 ac9831 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

전반적으로 작성 잘하셨습니다.
수고하셨습니다!

Comment thread Dockerfile
RUN npm ci --only=production

COPY prisma ./prisma
RUN npx prisma generate
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

여기는 prisma CLI이 없으면 실패 가능성이 있습니다.

그래서 보통 builder에서 이미 생성된 prisma Client를 그대로 복사하는 경우도 있는데요.

COPY --from=builder /app/node_modules/.prisma ./node_modules/.prisma
COPY --from=builder /app/node_modules/@prisma/client ./node_modules/@prisma/client

이런 식으로 작성해서 활용하기도 합니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants