깃허브에 새로 생긴 Approve!

최근에 회사에 입사하고 나서 소스리뷰에 대한 인식과 생각이 바뀐 상태에서 bitbucket 에 있는 approve기능을 좋아했다. 기능의 역활이 너무 명확하고 괜찮았기 때문이다.

approve기능은 단순했다. 어떤 직원이 PR(pull reqeust) 으로 해당 프로젝트에 코드를 입력시키면 그 코드를 모든 직원들이 읽어보고 그 코드가 난 괜찮다고 하면 누르는 기능으로 approve를 누르고, 이 approve가 어느정도 전체직원수에 맞게 승인이 떨어지면 PR을 merge 하는 방식이다.

 

이 기능이 깃허브에서 존재하지 않을까 하고 유심히 찾아보았다. 그런데 어제 새로운 문구를 발견했다. 아마도 기존부터 없었던 기능이 새로 생겼을때 그 기능을 알려주는 방식으로 설명이 나와있었는데, 거기 설명에 대한 내용을 읽어보거나 하지 못햇다. 하지만 Approve기능이 도입되었다고 나와있었던건 확실하다.

 

근데 아직 반쪽자리 기능이라고 느껴진다. Approve 하고 난다음 Unprove가 되어야 하고, 댓글을 남기면서 approve 할 수 있는 기능도 있던데 이를 지울 수 있는 기능이 아직 마련되어있지 않아 보인다.. =_=

 

빗버켓에서는 이렇게 현재 언프로브가 가능한 상태로 나오고 있으나 깃허브에서는

 

이미 approve 했음애도 불구하고 두번이상 연속 approve 가능하고..

 

지우거나 전혀 할 수 있는 상태가 아니였다. (디자인하나는 멋지네)
앞으로 부족한 기능이 많기 때문에 이 부분은 언제든지 해결되리라 생각하고 있고, 얼른 취소할 수 있는 부분도 클래임을 많이많이 넣어서 삭제하는 기능도 있었으면 좋겠다@_@

반쪽짜리 기능 리뷰 !