[TIL] 2021.11.06 최종 프로젝트 진행중

어제 push한 커밋 내용들이 꼬여서 커밋 로그를 정리했다

아래 그림은 정상적인 그림이다

각자 브랜치를 파서 push 하면서 계속 개발 하고, 일주일에 한번 코드리뷰를 한 다음 dev 브랜치에 합쳤다

합친 뒤 또 각자 브랜치에서 개발하고, 머지 후 첫 push하면 커밋 기록이 dev에서 뻗어나가야 한다

하지만 나는 dev에서 뻗어나간게 아니고 내가 만든 브랜치에서 뻗어나가는 것이였다

그래서 커밋해온 기록들을 머지한 직후로 되돌려놨다

# 커밋 기록을 간단하게 보기
git log --oneline

db7a3dd (HEAD -> board) feat: 댓글 좋아요/취소 기능 추가
6e16d7e feat: 댓글 삭제 기능 추가
be3221d feat: 댓글 조회 기능 추가
c94e256 update: BoardComment 엔티티 수정


# 커밋 기록 되돌리기
# "feat: 댓글 조회 기능 추가"가 가장 최근 커밋인 상태로 되돌아가기
git reset --hard be3221d


# 리모트 리포지토리에 반영하기
#+{브랜치이름} 중 '+'는 강제 push 명령어
git push origin +{브랜치이름}

강제 push를 하기 때문에 왠만하면 하지 않는 것이 좋다고 한다

 

---

위처럼 커밋 기록을 되돌려놓고 개발을 이전 브랜치에서 이어서 하려고 했다

그래서 로컬에서 dev 브랜치로 이동 후

git pull origin dev

명령어를 입력해서 땡겨왔따

그런 다음

git checkout {브랜치이름}

명령어를 입력해서 브랜치를 이동했더니

dev에서 pull해온 내용들이 이전 내용으로 바뀌는 것이었따..

그래서

git pull origin dev

명령어로 땡겨온 다음

dev 브랜치에서

git checkout -b {새 브랜치 이름}

명령어를 입력해 새 브랜치에서 개발을 이어서 했다

 

---

지금까지 코드리뷰 각자 브랜치에 합친 내용들을 pull해와 머지 후 dev에 push해왔다

하지만 오늘부터 방법을 바꿨다

pull request를 사용하려고 한다

각자 한 기능을 개발하고 각자 브랜치에 push할 때마다 pull request를 하기로 했다

멘토님이 pull request를 자주 하면 좋다고 하셨기 때문이다

기능별로 pull request를 하면 기능에 대한 코드들만 볼 수 있기 때문에 가독성이 좋아진다고 한다

그리고 2-3시간 잡아서 코드리뷰를 할 필요가 없어진다

pull request를 하면 멤버들이 코멘트를 남길 수 있다

아무런 설정을 안 해주면 pull request 한 뒤 바로 merge할 수 있다

하지만 우리 모두 코드를 한 번 씩 봐야하기도 하고, 코드에 문제가 있을 수도 있기 때문에

모든 백엔드 팀원들이 pull request 내용을 확인하고 코멘트를 달면서 approval해야지만 merge할 수 있도록

설정을 바꿔줬다

위의 사진은 내가 pull request한 화면이다

팀원들이 리뷰를 안 달았기 때문에 'Review required'라고 적혀있는거고,

모든 팀원들이 리뷰를 달면서 'approval'을 눌러줘야 머지를 할 수 있는데,

아직 아무도 리뷰를 안 달았기 때문에 'Merging is blocked'로 되어있다

 

 

 

issue 부분도 사용해봤다

이 기능은 자유게시판처럼 쓰면 된다고 한다

팀원들이 모두 확을 했으면 close를 눌러 내릴 수 있다

 

---

오늘은 개발만 하지 않았다

개발하는 시간은 많이 없었다

오전에 멘토님를 만나 피드백을 받고

개발 잠깐 하고 위에 적은 것들을 팀원들과 하루종일 얘기를 나눴다

깃허브의 커밋, 푸시만 해왔었는데 다른 기능도 있다는 것을 깨닫게 되었다

최대한 활용한다면, 더 편리하게 커밋할 수 있을 것 같다