오늘은 멘토님와 대화중에 나온 git pull Reqeust 에 대해 알아보도록 하겠습니다.

pull Request 은 무엇인가 ?

물론 이전 포스트에 대해 이에 대한 설명을 써놓았지만 간단하게 말해서

내가 작성한 코드를 승인 받고 merge 하는방식 

그럼 왜 코드를 승인을 받아야 하는가

• 사람이 작성한 코드라 분명 어디에 실수가 있을수도 있다.
• 이게 최선인 코드일까 ?
• 코드리뷰 및 코드 개선으로 일을 두번 안하게 위해서

전 아무래도 맨 마지막 부분이 공감이 가더라구요

그럼 pull reuquest 은 어떻게 만드는가

이전 Android stuido 에서 사이드 프로젝트를 했을시 이미 pull Request을 하고 또다른 기능을 pull Request 을 시도 한적이 있습니다.

그럴때 git 에서는

이미 pull Request 가 있는데 뭘 또만듬? 이전 코드 merge 하고 해 

저는 깃을 코드 저장만 해서 이런상황에 매우 당황 했습니다.

그것을 멘토님에게 물어보니

멘토님 : 아 ㅎㅎㅎ... pullRequest 은 그렇게 하는게 아니에요 . 
주로 branch 을 따서 세분화 해서 pullRequest을 만들어요 

예를들어 큰 주재인 회원가입을 생각해볼께요 

그럼 회원가입을 만들때 필요한 issue 가 있겠지요? 

예를 들어 
1.이미 회원인지 파악하고 
2.nickName이 중복이 안되는지 확인하고 
3. 회원가입을 하겠지요 

이럴때 branch 을 나눠서 주로 pull Request을 해요 

음 저로써는 꽤 신기한 경험 이었습니다.

pull Request 검증

자 그럼 제가 Pull Request을 만들어서 사수분께 검토요청을 했다고 가정해봅시다.

예를 들어

• 회원 중복 검사
• 회원 nickName 검사
• 회원가입 로직

이렇게 pullRequest을 만들어서 검토 요청을 해봤다고 생각을 해볼께요

근데 사수분도 일이 많겠지요? 뭐 일이 없을때는 천천히 코드를 봐주면서 피드백을 해주실수도 있지만

대부분 기업체에서는 수익창출을 해야하기 떄문에 모든 인력을 놀게 안놔둡니다.

그래서 사수분도 바쁜데 이 많은 코드를 본다.

아마 좋은 소리는 안나올거 같습니다.

그럼 해결법은?

1. 회원가입 branch 생성
2. 회원가입시 필요한 issue 마다 branch을 따서 만듭니다.
3. 코드가 짧음으로 읽는 시간이 짧아져 빠른 피드백 완성

이런식으로 하는것을 멘토님께서 추천을 해줬스빈다.

전 보통 git을 코드 저장용도로 만들었는데 꽤나 신기하네요

그럼 오늘도 읽어 주셔서 감사합니다

-피드백와 비판은 언제나 환영입니다-