Skip to content

42-CPP-GIT/CPP_GIT

Folders and files

NameName
Last commit message
Last commit date

Latest commit

 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 

Repository files navigation

🔥 CPP_GIT 🔥


💪 스터디 목표

CPP와 GIT을 같이 함께 배우자!

📅 스터디 기한

11월 15일까지!

🤝 Ground Rule

  1. C언어 La Piscine처럼 각자 코드로 진행을 하기! (각자 이름으로 된 Branch 에서 작성)
  2. 한 파트를 다 했을 경우, 오프라인으로 팀원에게 평가를 받기 (최대 2명 가능!)
  3. 평가 OK를 받을 경우, PR을 날리기
  4. PR을 날릴 경우, 최대한 본인이 배운 것들 정리해서 작성하기
  5. MERGE는 최소 스터디원 3명이 Code Review를 한 후, APPROVE를 해야 가능하다.
  6. 출처는 꼭 표시

⁉️ FAQ

Q. 실제 42서울 평가는 언제 받을 수 있나요?
A. 오프라인으로 팀원에게 평가 받으면 그 뒤 부터는 가능합니다. (PR 올리고, 코드리뷰 받고 42서울 평가 받는 것을 권장하긴 합니다.)

Q. PR이 Merge(closed)되어서 Code Review를 하지못했어요. Review를 하고 싶으면 어떻게 하죠?
A. PR이 Closed되어도 Review는 여전히 달 수 있습니다. 끊임없이 질문해보세요!

Q. ISSUES는 쓰지 않나요?
A. 각 Module을 풀 때, 알아갔던 것들을 ISSUES에 작성하고, PR에는 ISSUES번호를 적어서 설명을 하셔도 좋습니다.

Q. PR을 했고, MERGE는 아직 안했는데 수정사항이 생겼어요. 혹은 피드백을 받고 수정하고 싶어요. 다시 PR을 올려야하나요?
A. 아닙니다. 그 Branch에서 다시 코드를 수정하고, commit->push를 하면 그 PR 창 하단에 바뀐 정보들이 뜨고, 코드가 바껴서 올라갑니다. 다시 PR 올리실 필요 없습니다.

Q. PR을 해서 Merge를 끝냈는데 수정하고 싶은 사항들이 있어요.
A. 이럴 때는 다시 PR 올려주셔도 좋습니다.

🧚‍♀️ PR 시, 작고 귀여운 룰

Screen Shot 2022-10-24 at 8 57 44 PM

들어오면 아시겠지만 PR할 때 템플릿 추가해놨습니다.

PR할 시, 제목은 [Intra ID] CPP0X 적어주시면 될 것 같습니다.
Reviewers는 자동으로 3명으로 적용될 예정입니다.
Assignees는 본인을 해놓으시면 됩니다.

PR 템플릿을 보면, PR은 오프라인 평가를 1회 받은 뒤 올려야하니 평가해주신 분 인트라 id를 적어주시면 됩니다.
이번 과제에 알아야하는 개념들은 짧게 어떤 것들이 쓰였는 지 적어주시면 됩니다.
이번 과제를 통해 알아낸 것들은 본격적으로 어떤 것을 배우셨는 지 적어주시면 됩니다.

맥을 쓰시기에 스샷, 화면 녹화 또한 쉬울테니 이런 것들도 적어주시면 알기 쉬울 것 입니다.
수정사항(optional)은 pr을 올렸음에도 뭔가 수정하고 싶은 것이 생겼다하시면 올리시면 됩니다.
간단한 이야기이지만 PR을 올리면 톡방에 한번 이야기를 해주시면 빠르게 리뷰할 수 있을 것 입니다.

👨‍👩‍👧‍👦 스터디 멤버