제출된 코드가 프로젝트의 설계 방향과 맞지 않을 수 있어.


그래서 직접 메인테이너가 고치거나 수정을 요청할 수도 있는거고.



그 레포 메인테이너가 그랬지? "구현 아이디어 일부를 반영했다" 고.


그러면 적어도 그 사람이 기여한 커밋 로그나 이런저런 흔적은 남겨야 예의겠지.


GPLv3로 프로젝트를 공개할정도로 그렇게 애착이 큰 물건이라면 더더욱 그렇지 않겠어?



그런데 실제로는 어떻게 했지?


8월 27일 저녁 8시에 구현을 포함한 커밋들과 같이 PR이 올라왔는데


이틀전 (9월 1일 19시 20분)에 아무 말도 없이 마스터 브랜치로 메인테이너가 직접 커밋했어


그리고 9월 2일 23시 48분에 프리릴리즈를 걸었네?


그걸 보고 PR 올린 애가 빈정이 상했나봐. 2시간쯤 뒤인 1시 23분에 PR 클로즈하고 브랜치를 날렸어.


그래놓고 오늘 아침 9시 27분에 메인테이너가 코멘트라고 달아놓은게 이거야.


안녕하세요, 시간 내어 구현해 주셔서 감사드립니다. 기능과 구조에 충돌이 있어 아쉽게 머지하지는 못하였으나 구현 아이디어 일부를

1b72cab 에 반영하여 v1.1.0 (19)로 선공개하였습니다. 시간이 되신다면 테스트와 피드백 부탁드립니다. 감사합니다.


1.1.0 이라고 올려놓은 프리릴리즈를 보면 이미 올라왔던 이슈가 같이 해결된 PR인거 같더라고.


그 이슈가 4월 19일에 올라왔던걸 보면 그냥 확인만 하고 몇달동안 신경 못쓰던 이슈겠지. 그럼 더더욱 이러면 안됐어.


정 수정 요청하기 뭐하거나 반영하기 까다로운 문제여서 직접 커밋해야 했다면


https://gall.dcinside.com/mgallery/board/view/?id=github&no=51487&page=1

PR 관련해서 너무 논쟁이 과해진것 같은데..당연하지만 기여자는 프로젝트 전체의 기능, 히스토리를 모를수밖에 없음컨텍스트 정보가 적으니 자신이 쓰다가 발견한 문제에 집중한 경향이 있게되지따라서 기존 기능 혹은 로드맵 상의 충돌이 있을범직도 함코딩과 커밋 스타일은gall.dcinside.com



여기서 말마따나 Co-Authored-By 같은 기능이라도 쓰던가 했어야지.


그걸 몰랐으면 적어도 커밋 revert 로그 박고 직접 작업한다음에 머지한다는 방법도 있었잖아?


커밋 메세지에 PR 번호 딸랑 걸어놓으면 그게 눈가리고 아웅이 아니고 뭐겠어?




알고도 이랬으면 그냥 커밋메세지 오염(?) 시키기 싫어서 이슈 해결 날로먹을겸 아이디어 도둑질 친거고


몰랐더라도 생각이 짧았다고 밖에는 설명할 방법이 없지





내가 참으로 아쉬웠던건,


PR 처음 달린 시점부터 자기가 직접 커밋할 때까지 며칠의 시간이 있었는데


이런 생각을 한번도 안해볼 정도로 급하게 처리했어야 했던 이슈였을까? 라는거.





참조:


https://archive.is/egwIH

archive.isarchive.isarchive.is


https://archive.is/BJeHn

Caps Lock 고정 관련 · Issue #3 · kiding/SokIM보관 2023년 9월 3일 13:52:40 UTCarchive.is