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

.dockerignore 파일 업데이트 #64

Merged

Conversation

yuminn-k
Copy link
Contributor

🔍 이 PR로 해결하고자 하는 문제는 무엇인가요?

  • Docker 이미지 빌드 시 불필요한 파일들이 포함되는 문제를 해결하고, 필요한 파일만 포함되도록 .dockerignore 파일을 업데이트했습니다.

✨ 이 PR로 주요하게 바뀐 점은 무엇인가요?

  • .dockerignore 파일을 업데이트하여 필요한 파일만 포함되도록 수정했습니다.
  • .env, go.mod, go.sum, cmd/, internal/, pkg/, main.go 파일만 포함되도록 설정했습니다.

🔖 주요 변경 사항 외에 추가로 변경된 부분이 있나요?

  • 없음

🙏🏻 리뷰어가 특히 봐주었으면 하는 부분은 무엇인가요?

  • .dockerignore 파일 설정이 최적화되었는지 확인 부탁드립니다.

🩺 이 PR로 테스트나 검증이 필요한 부분이 있나요?

  • Docker 이미지 빌드 테스트가 필요합니다.

📚 관련된 Issue나 Notion, 문서

🖥 작동하는 모습

스크린샷이나 녹화된 비디오, 또는 gif를 추가해서, 리뷰어가 변경점을 이해하는 데 도움이 되도록 해주세요.

📌 PR을 작성할 때의 주의사항

  • 리뷰어는 코드 리뷰 시 좋은 코드의 방향성을 제시하지만, 코드 수정을 강요하지 않습니다.
  • 리뷰어는 좋은 코드를 발견하면 칭찬과 격려를 아끼지 않습니다.
  • 리뷰는 특별한 경우가 아니라면, 리뷰어로 지정된 시점부터 3일 이내에 완료해주세요.
  • 댓글 작성 시 Prefix에 P1, P2, P3을 작성해주시면, 담당자가 댓글에 대해 더 명확히 대응할 수 있습니다.
    • P1 : 반드시 반영해주세요 (Request Changes) - 문제가 발생하거나, 취약점이 발견된 경우 등.
    • P2 : 반영을 적극적으로 검토해주세요 (댓글).
    • P3 : 이런 방법도 있을 것 같다는 사소한 의견입니다 (Chore).

- Updated .dockerignore to ignore all files except necessary ones
- Ensured .env, go.mod, go.sum, cmd/, internal/, pkg/, and main.go are included

Related to ryokushaka#37
@yuminn-k yuminn-k added ⏰ P0 반드시 해결해야 하는 중요한 이슈나 문제 (예: 버그, 취약점 등) 🐞 BugFix Something isn't working 🆗 safe CI workflow 동작 관련 🌏 Deploy 배포 관련 🔨 Refactor 코드 리팩토링 ⚙ Setting 개발 환경 세팅 labels Oct 14, 2024
@yuminn-k yuminn-k self-assigned this Oct 14, 2024
@yuminn-k yuminn-k merged commit 69cd1b1 into ryokushaka:main Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 BugFix Something isn't working 🌏 Deploy 배포 관련 ⏰ P0 반드시 해결해야 하는 중요한 이슈나 문제 (예: 버그, 취약점 등) 🔨 Refactor 코드 리팩토링 🆗 safe CI workflow 동작 관련 ⚙ Setting 개발 환경 세팅
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant