첫 PR 날려봤다

NFS 관련 공식문서를 읽다가 예시를 보기 위해 링크를 눌렀는데

잘못된 링크였다.

 

확인해보니 master 브랜치인데 링크는 main 브랜치로 되어있던 것이 문제였다.

링크를 고쳐달라고 문의를 보내려고했는데, 그런 방식이 아니라 kubernetes/website라는 github 프로젝트로 관리되는 거였다.

 

그래서 깃헙 코드를 직접 확인해보니

{{<param "githubbranch" >}}를 통해 파라미터로 브랜치를 받고있었다.

그래서 처음에는 이 파라미터를 넘겨주는 값을 main에서 master로 바꾸면 되겠다 싶었지만,

도대체 어디서 넘겨주는지를 알 수가 없었다.

 

그런데 다행히 똑같은 이슈의 PR들이 이미 있었다.

답변이 말하기로는 파라미터 자체는 해당 프로젝트의 default 브랜치 값을 자동적으로 주어서 문제가 없다고 한다.

그 프로젝트의 default 브랜치가 이름이 바뀌어서 문제가 생긴 것으로 보인다. 결국 하드코딩 하란다!

 

그래서 생애 첫 PR을 시도해보았다.

fork한 뒤 branch를 따서 수정을 시작했다.

사실 수정이랄 것도 없다.. 그냥 하드코딩으로 링크 주소를 바꾸는 거니까..

그래도 일일히 찾아서 수동으로 바꾸긴 귀찮으니 grep과 sed를 이용해 한번에 바꾸었다.

$ grep "{{< param \"githubbranch\" >}}" * -rl | xargs sed -i 's/{{< param \"githubbranch\" >}}/master/g'

그리고 커밋을 한 뒤, 생애 첫 PR을 날렸다!

 

그랬더니 PR을 관리해주는 봇이 뭔가를 쫘라락 달아줬다.

CLA 서명을 하라고해서 CLA 서명을 했다.

그리고 이걸 그냥 기다려야 되는건지 /assign @gochist를 하란건지 이해가 잘안가서

그냥 /assign @gochist를 해서 담당자에게 노티한 상태이다.

 

첫 PR이라 내가 제대로 한 건지 모르겠다...

나중에 경과를 지켜봐야겠다.