원래 오픈소스 기여가 Issue -> PR -> Change Request -> Fix -> Review again -> merge 이 과정 아닌가??
그래서 Change request 들어온거 다 고친 다음에 review 해달라고 걸었는데 이 씨발 리뷰하는 방법을 모르는거 같음 그냥
rebase, ruff도 모르길래 진짜 설마설마 했는데 차단 당했네 뭔 씨발,,,, 리뷰할 줄도 모르는 새끼한테 신나서 기여해주겠다고 혼자 그냥 난리를 쳤네
아 진짜 허무하다 이래서 오픈소스는 무조건 인증된 대기업에만 해야하는데 하...
링크좀
https:_//github.com/HomebrewML/HeavyBall/pull/52
아니 나 진짜 뭘 잘못한건지 모르겠다 그냥.. 댓글들 막혀서 못보는 내용 많긴 할텐데, 커밋로그 위주로 봐주셈
심지어 커밋로그 지저분해져서 rebase 돌렸더니만 왜 PR 여러개 쓰냐고 한다 진짜 환장하겠음,,
어차피 squash 할건데 커밋로그 지저분한게 뭔상관이냐 있는 pr 위에서 계속 커밋을 쌓아야 바뀐거 계속 트래킹하고 리뷰가 되지
그렇다고 PR 2개 열었다는게 그렇게 잘못한거야?? 그리고 애초에 리뷰하는 놈이 rebase를 모르는거 같던데.. 커밋메세지 지저분한거 봐봐
당연히 잘못한거지 시발 니 맘대로 pr닫고 rebase해서 오픈하면 시간들여 리뷰한거 사실상 날아간거랑 다름이 없는데
나중에 main에 squash 되면 pr 안에서 커밋메세지 다 사라지는데 도대체 그걸 왜 신경씀?
리뷰한게 왜 날아가냐.. change request 들어온거 전부 수정한 다음에, rebase 했다니까?? 초기화 한게 아니라 고치라는거 잡은 뒤에 열었다는거야
rebase 없이 pr위에 쌓아야 니가 추가로 고친걸 확인하지 이걸 아직도 이해 못함???
대충 만줄짜리 pr 수정한거 다 리뷰하고 대충 10줄 정도 고치라했는데 위에 10줄 수정한 커밋들만 보고 확인하는게 쉽겠냐 다시 rebase된 만줄짜리 검토하는게 쉽겠냐 회사에서 저지랄하면 싸우자는거밖에 안됨
아니 review가 rebase를 전혀 모르길래 다듬어주려고 했는데 그럼 커밋을 10개 20개 계속 쌓으라고??
너 말도 맞긴한데, 이번에는 쫌 다르다. 그래서 너 말처럼 커밋마다 잘게 쪼개서 수정해줬는데, 그냥 그걸 읽을줄 몰라 진짜로
진짜 링크 봐봐 그냥 커밋로그 확인 안하고, "넌 수정하지 않았다" 이 소리만 하고있음.. 중간중간 내가 수정한 커밋 다 있거든?? 링크 봐봐.. 보고도 그렇게 얘기하면 뭐라 안할게
51번 pr 보니 pre-commit 수정으로 오지게 커밋쌓은건데 그러면 추측컨데 큰 feature 구현 없이 단순 스타일/리팩토링 정도일거고 그러면 리뷰어 입장에선 feature pr 열어놓고 수정안하고 뭐하는거지 생각들거같은데
바뀐 파일들 목록도 보니 pr 스코프랑 상관없는 전체파일에 무지성으로 ruff 돌려서 쓸데없이 리뷰하기 힘들게 만든거 같은데 리뷰어가 쌍욕 안박은거만 봐도 보살이다
우선 48 PR 보면 알겠지만, 리뷰어는 pre-commit 을 전혀 모르는 상태였음. 다른 contributor가 yaml 써놓긴 했지만 불완전했고, 코드베이스 전체가 pre-commit이 적용되지 않은 상태였음
그리고 PR을 52에서 다시 열었던게, 그래서 너 말대로 리팩토링, pre-commit 관련 내용을 하나로 뭉개야 리뷰하기도 편할거 같다는 생각이였음
당연히 메인테이너가 저런 기술 모를수 있고 너가 의욕넘치고 열정있는건 알겠는디 근데 그걸 할거면 새로 이슈를 열어서 메인테이너한테 제안하고 해야지 feature 브랜치에 짬때리면 되겠냐
아 그런가.. 난 그냥 하는 김에 같이 리팩토링하면 좋겠다고 생각했어. 그리고 또 이해가 안가는게, 그 PR이 너무 지저분해져서, 이전 PR에서 수정사항 말한 내용들 잡고 다시 리뷰 요청 했거든??
근데 다 고치고 했더니 95% 수정이 안됨 ㅇㅈㄹ하길래 하나하나 고친거 보여주니까 그제서야 리뷰 시작하더라... 이것도 내가 잘못한거야??
리뷰를 할거면 PR 안에서 커밋 쌓일 때마다 보면서 하나씩 resolve 해야하는데 그것도 아니고, 너무 비일관적인 느낌이였어
애휴 그래도 고맙다 내가 괜히 의욕만 앞서서 이것저것 했나보네.. 요즘 여기저기서 PR 받아주니까 너무 거만했던 것 같음 쩝
다른 pr들을 안봐서 잘은 모르겠다만 너말대로 협업에 익숙치 않았다면 저런 rebase된 pr 보고서 왜 거의 똑같은 pr을 올리지 했을거같긴허다 그리고 상대가 뭔가 실수했다 싶음 새 pr 닫고 예전 pr 다시 오픈해서 안내했을수도 있음직한데
욕봤다 뭐 액땜했다 생각하고 다른데 기여하면 되지
엉.. 논문 구현하느라고 거의 일주일 갈아넣어서 내가 너무 예민했나봄.. 차근차근 PR 날려야 하는데, 괜히 ruff 끼워넣은게 문제 같다. 다음에는 하나씩 해볼게 고맙다
너가 억울할 수도 있는데 원래 이런 작은 오픈소스는 마갤이랑 비슷하다 주딱 맘에 안들어서 차단당하면 어쩔수 없음
ㅇㅋ 다음에는 더 조심할게 고마워