오랜기간 코드리뷰를 거친 후 merge를 해야 할때 참고할 사항들

코드리뷰를 모두 마치고 올렸는데 git conflict가 발생하였다.

이전에 계속해서 했던 작업이지만 오랜만에 하니 기억이 안나 다시 정리하도록 하겠다.

먼저 명령어의 실행 위치는 원본 소스가 아닌 코드리뷰를 받는 소스이다.

해당 소스에서 원본 소스를 머지해야 원본에서 업데이트 된 코드가 리뷰를 받아 적용이 되게 된다.

git merge develop

해당 명령어를 내리면 다음과 같이 충돌이 발생하는데 이걸 모두 클릭해서 충돌을 해결해 주어야 한다.

충돌을 모두 해결하고 나면 

 

해당 변경 사항들이 commit에 올라가게 된다.

 

여기서 한번 더 수정사항이 맞는지 체크를 한 후 수정사항이 맞다면 push를 해서 수정된 코드를 반영해주면 해결이된다.

 

이떄 또한 주의할 점은 push를 하고 나서 한번 소스를 실행시켜 보는 것이다. 소스를 실행시켜봐서 정상적으로 충돌 없이 작동해야지 완전히 충돌이 해결된 것이기 때문에 vs code에서 못잡는 부분까지 실행을 시켜서 잡아서 충돌을 해결해주어야 한다.

 

또한 이번에 오랜기간 반복 코드리뷰를 받다보니 반영시에 74개의 commit을 한번에 머지해야 했다. 이때는 squash merge를 사용하면 일자별로 나누어서 깔끔하게 머지가 된다. 머지를 올릴때 만약 이번과 같이 오랜기간 수정을 거쳐 머지를 하면 squash merge를 사용하도록 하자!