Skip to content

[이창호]sprint11#167

Merged
ac9831 merged 1 commit into
codeit-bootcamp-nodejs:이창호from
dlckdgh0523-lgtm:sprint10
Apr 1, 2026
Merged

[이창호]sprint11#167
ac9831 merged 1 commit into
codeit-bootcamp-nodejs:이창호from
dlckdgh0523-lgtm:sprint10

Conversation

@dlckdgh0523-lgtm
Copy link
Copy Markdown
Collaborator

요구사항

기본

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

심화

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

주요 변경사항

스크린샷

image

멘토에게

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

@dlckdgh0523-lgtm dlckdgh0523-lgtm changed the title feat:sprint11 [이창호]sprint11 Apr 1, 2026
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 .github/workflows/ci.yaml
--health-retries 5

env:
DATABASE_URL: postgresql://panda:panda1234@localhost:5432/panda_market_test
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.

하드코딩 되어 있는 것중 민감한 정보는 따로 .env로 빼는것을 추천드립니다.

Comment thread Dockerfile
@@ -0,0 +1,20 @@
FROM node:20-alpine
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.

추후 멀티 스테이지 적용을 해보시는 걸 추천드립니다.
싱글 스테이지와 차이점을 확인하고 왜 멀티 스테이지를 구성해야 하는지 아는 것이 중요합니다.

@ac9831 ac9831 merged commit 8485739 into codeit-bootcamp-nodejs:이창호 Apr 1, 2026
1 check passed
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