공부/Kubernetes

첫 PR 성공적으로 merge!!

토고미 2021. 9. 24. 03:47

이 글에서 이어진다.

추석 연휴 전에 날린 PR이 우여곡절 끝에 merge 되었다.

그 와중에 사소한 이슈가 좀 있었다.

가장 먼저 알게 된 것은 assign은 어지간하면 내가 하는 게 아니라는 것!

어차피 봇이 알아서 리뷰어를 할당해주고 하기때문에 난 그냥 PR만 올리면 된 거였다.

 

아무튼 그래서 빨리 될 줄 알았는데..

 

내가 main 브랜치로 merge로 요청을 한 게 문제가 됐다.

한국어 작업을 하는 브랜치로 targe branch를 변경해달라고 하셨다.

너무 친절히 알려주셔서 정말 감사했다.

 

그런데 서지훈님께서 "혹시 이런 현상~"이라고 말씀해주신대로 문제가 발생했다.

내가 main 브랜치에서 분기를 쳐서 작업을 했다 보니, 한국어 브랜치로 merge 하는데 문제가 생긴 것이다.

 

어찌저찌 한국어 브랜치를 베이스로 다시금 작업해서 새로운 PR을 날렸다.

 

손석호님의 환영인사와 함께 approve까지 순식간에 되었다.

 

마침내 내 커밋이 merge되었다!!!

그런데 24시간이 안 지나서 그런 건지, main 브랜치가 아니여서 인지는 모르겠지만 contributor에서는 찾아볼 수가 없었다...

 

겨우 오타 수정이기는 하지만, 그래도 나름 오픈소스에 기여해서 뜻깊었다.

나도 이제 오픈소스 컨트리뷰터다!!

 

내가 누구?

"Kubernetes docs Contributor"