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

github 작업 가이드 🦮 #45

Open
wonslee opened this issue Feb 10, 2023 · 0 comments
Open

github 작업 가이드 🦮 #45

wonslee opened this issue Feb 10, 2023 · 0 comments
Labels
봐주세요 announcement

Comments

@wonslee
Copy link
Owner

wonslee commented Feb 10, 2023

컨벤션

  • 브랜치 이름 : 대주제/소주제 (e.g. algorithm/sort )
  • 커밋 메세지 : [대주제] 이번에 다룬 주제
  • PR 및 일정 issue 제목 : 커밋 메세지와 동일. [대주제] 이번에 다룬 주제

branch 전략

  • master와 각 멤버가 작업할 대주제/특정 주제 branch들로만 쪼개짐. PR merge는 master을 향해서만 실행

image

일정 등록 및 close

  1. 진행할 주제를 PR제목과 동일한 제목으로 issue 생성
    image

  2. PR 생성시 Development 섹션에서 해당 issue를 연결
    image

  3. PR에서 몇주차 스터디인지 Milestone 붙여주기
    image

  4. PR merge시 자동으로 issue까지 자동으로 close됨. merge는 PR 생성한 사람이 함

리뷰와 승인 과정

  • 리뷰
    멤버들은 서로의 PR에 가서 files changed 내용을 보고, 오류가 있지는 않은지 검토합니다.
    내용 수정이 이뤄졌다면, 어떤 라인의 어떤 내용이 바뀌었는지 얘기해보면서 토론을 해보면 좋을 것 같아요! 😆

  • 승인과 머지
    나머지 멤버들 모두의 review 및 approve를 받아야 merge 가능.
    왠만하면 되겠지만, 만약 컨벤션이 지켜지지 않았거나 내용에 너무 큰 오류가 있는 경우 승인되지 않습니다 (해주지 맙시다! ).

conflict시 대피요령

  • 문제가 생기면 언제나 SOS 요청합시다. (해결사 @koownij 씨와 상담 🙏 )

git_conflict drawio

@wonslee wonslee added the documentation Improvements or additions to documentation label Feb 10, 2023
@wonslee wonslee added 봐주세요 announcement and removed documentation Improvements or additions to documentation labels Feb 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
봐주세요 announcement
Projects
None yet
Development

No branches or pull requests

1 participant