Skip to content

Conversation

@ParkGyeongTae
Copy link
Contributor

#️⃣ Issue Number

📝 요약(Summary)

  • 도커 관련 파일의 구조 변경

💬 To Reviewers (선택)

  • docker compose up -d

PR Checklist

  • N/A

reference) How to Code Review

  • 따봉(👍): 리뷰어가 리뷰이의 코드에서 칭찬의 의견을 남기고 싶을 때 사용합니다.
  • 느낌표(❗): 리뷰어가 리뷰이에게 필수적으로 코드 수정을 요청할 때 사용합니다.
  • 물음표 (❓): 리뷰어가 리뷰이에게 의견을 물어보고 싶을 때 사용합니다.
  • 알약 (💊): 리뷰어가 리뷰이의 코드에서 개선된 방법을 제안하지만 그것의 반영이 필수까지는 아닐 때 사용합니다.

@ParkGyeongTae ParkGyeongTae self-assigned this Sep 18, 2025
Copy link
Collaborator

@ehddnr301 ehddnr301 left a comment

Choose a reason for hiding this comment

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

@ParkGyeongTae 항상 코드 구조화 관련해서 좋은 제안 감사합니다!!

❗❓ 저도 리뷰를 위해 찾아보다가 발견했는데 build context 관련해서 현재 build context가 docker 경로 보다 상위 경로여서 .dockerignore 는 적용이 되지 않을듯 합니다. 제가 이해한 부분이 맞다면 변경 요청드립니다! (아니라면 편하게 말씀 부탁드려요!)

💊 ChatGPT에게 물어봤을떄는 Dockerfile.dockerignore 를 쓰라고 말해주고 있습니다

@ParkGyeongTae
Copy link
Contributor Author

@ehddnr301 리뷰 감사합니다! 수정했습니다~!

Copy link
Collaborator

@ehddnr301 ehddnr301 left a comment

Choose a reason for hiding this comment

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

👍 고생하셨습니다아!!!

@ParkGyeongTae ParkGyeongTae merged commit e72f426 into master Sep 21, 2025
1 check passed
@ehddnr301 ehddnr301 deleted the 154-restructure-docker-directory branch October 11, 2025 11:04
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.

3 participants