Skip to content

Mission9#239

Open
JunoLee1 wants to merge 28 commits intocodeit-bootcamp-nodejs:mainfrom
JunoLee1:mission9
Open

Mission9#239
JunoLee1 wants to merge 28 commits intocodeit-bootcamp-nodejs:mainfrom
JunoLee1:mission9

Conversation

@JunoLee1
Copy link
Collaborator

요구사항

기본

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

심화

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

주요 변경사항

스크린샷

image

멘토에게

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

@rdd9223 rdd9223 self-requested a review November 19, 2025 12:51
Copy link
Collaborator

@rdd9223 rdd9223 left a comment

Choose a reason for hiding this comment

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

지금 mission9/node_modules가 ignore되지 않아서 merge가 불가능한 상태입니다! 꼭 dist, node_modules, coverage 등 실제 git으로 관리되지 않아도 되는 큰 파일들은 모두 ignore처리해주세요.

추가로 테스트부분은 구현하지 않으신 것 같아요. 시간이 부족해서인지, 아니면 개인적인 이유에서인지 멘토링시간에 이야기해보죠!

node_modules ignore이후에 merge 해주세요.

Comment on lines +4 to +8
mission2
mission3
mission4
mission6
mission8
Copy link
Collaborator

Choose a reason for hiding this comment

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

지난 미션은 왜 ignore 처리해두신건가요? 굳이 ignore 처리 안해도 될 것 같은데, 아마 git을 사용하는데 어려움이 있으신 것 같아요. 이건 멘토링 시간에 이야기 해보죠!

Comment on lines +9 to +10
mission9/dist
mission9/generated/prisma
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기서 node_modules를 ignore처리 해주세요.

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.

4 participants