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

feat: api, data, email, common CODEOWNERS 추가 #25

Merged
merged 1 commit into from
Jun 8, 2024

Conversation

belljun3395
Copy link
Collaborator

🎫 연관 이슈

resolved: #24

💁‍♂️ PR 내용

  • api, data, email, common CODEOWNERS 추가

🙏 작업

  • api, data, email, common CODEOWNERS를 추가하였습니다.

🙈 PR 참고 사항

📸 스크린샷

🤖 테스트 체크리스트

  • 체크 미완료
  • 체크 완료

@github-actions github-actions bot added the feature 새로운 기능을 만들 때 사용됩니다 label Jun 8, 2024
/email @hun-ca @belljun3395

/common @hun-ca @belljun3395

/config @hun-ca @belljun3395

/docs @hun-ca @belljun3395
Copy link

Choose a reason for hiding this comment

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

이 코드 패치의 커널 부분이 표시되지 않아서 코드 자체에 대한 리뷰를 제공하기는 어렵습니다. 하지만, 보여진 부분에 따르면 다음과 같은 사항을 확인할 수 있습니다:

  1. @hun-ca@belljun3395 두 명이 모든 디렉토리 (src, api, data, email, common, config, docs 등등)에 대한 코드 리뷰어로 지정되었습니다. 이는 혼동을 줄이고 코드의 일관성을 유지하는 데 도움이 됩니다.

  2. 하나의 주의 사항은 두 개발자가 모든 영역의 전문가라고 가정하는 것입니다. 강점과 취약점을 고려하여 팀의 다른 멤버들도 코드 리뷰에 참여하도록 하는 것이 좋을 수 있습니다.

  3. 작업 부하가 너무 많거나 전문 분야가 너무 광범위한 경우, 두 사람 모두에게 일부 범위를 할당하는 것이 좋습니다. 그렇지 않으면 코드 리뷰 과정이 지연될 수 있습니다.

  4. 또 지금의 코드에서는 오류를 찾을 수 없습니다. 그러나, 실제 소스코드가 제공된다면 더 구체적인 코멘트나 피드백을 드릴 수 있습니다.

@belljun3395 belljun3395 merged commit b83f682 into main Jun 8, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 새로운 기능을 만들 때 사용됩니다
Projects
None yet
Development

Successfully merging this pull request may close these issues.

모듈에 맞게 코드 오너를 수정합니다
2 participants