[Git] pull request 정의

Pull request

pr 관련 궁금증을 해소하다가 PR의 개념에 대해 질문을 받게 되었다.

나는...

PR 이 뭐냐에 대한 나의 대답은
"main이나 master branch 에 merge 하기 전에 코드 작성 후 리뷰, 리팩토링 등 하는것" 이었다.

하나부터 열까지 모두 틀린 대답이었다.

어디가 틀렸을까

1. main 이나 master branch
   -> Timotae 프로젝트의 branch 전략이 github-flow 이기 때문에 가능한 대답이다. 만일 git-flow 나 gitLab-flow 였다면 develop 가 될 지, production 이 될 지 모르는 일이다.

2. merge 하기 전에 코드 작성 후
   -> merge 전 코드 작성을 하는 것 자체는 맞다고 볼 순 있지만, PR 에서의 코드 작성 단위는 branch 이다. 처음 개념을 말할 때의 중심이 리뷰와 리팩토링에 치우쳐 있어서 나온, 틀리진 않지만 정답도 아닌 애매한 대답이다.

3. 리뷰, 리팩토링을 하는 것
   -> PR의 목표는 리뷰나 리팩토링이 아니다. 부가적인 목표가 될 순 있겠지만, 궁극적인 목표는 서버의 코드나 데이터를 pull 하기 위해서, 그리고 로컬에 있는 코드들을 상단의 branch 에 merge 하기 위한 것이다. 따라서 다 제쳐두고 리뷰와 리팩토링을 위해서 한다 ! 라는 대답은 정답이 될 수 없다.

확실히

하면 할 수록 어려운게 느껴진다. 아직 제대로 시작도 안 한거 같은데 벌써부터 헷갈리는게 나오고 잊어버리는 게 많아진다. 시간을 더 투자해야겠다 !