풀리퀘스트를 받는 임의의 여러 명이 내 코드를 보고
내 코드의 좋은점 나쁜점을 평가해서 개선하도록 유도하는게 가능한건가 의문...
내 코드의 앞뒤 문맥, 전제 설계방향을 아는건 오로지 나 혼자인데, 그속에서 발생하는 문맥처리 없이
그냥 풀리퀘스트로 올리는 코드들만 보고 그게 유추가 가능한가? 실행을 하는 것도 아니고...
뭐, 컨벤션같은걸 안지킨거야 개선가능하겠지만... 그 이상이 가능한가?
예를 들어, 내가 구현한 내용이 구체 클래스로 바로 접근하는 방향으로 했는데,
코드 리뷰어가 나보다 윗급 직책이고 인터페이스를 중요하게 여겨서, 구체 클래스 바로 쓰지말고 인터페이스 쓰세요해서
고치는게 의미가 있나?
궁금해.
난 이런 프로세스를 안겪어봐서 알고싶음
그래서 유닛테스트 짜오라고 강제하는 프로젝트들도 있음
플리퀘 올리면 유닛테스트 결과 봇이 가져오고 리뷰어는 테스트 코드랑 수정된 부분 보고 판단하고
유닛테스트 짠다고 해서 코드리뷰를 안받는건 아니자너
아하 그런 방법으로 하면 좀 낫겠네
계속 질문해도 되나? 이렇게 코드 리뷰를 한다고 했을 때, 코드 리뷰에 하루에 얼마나 시간을 쓰는지 궁금
기능 크기 따라 다르지 근데 리뷰하는사람이 잘 모르면 동작하나싶다 하면 lgtm 날릴 수도 있고 다른사람 부를수도 있음 사람차임
여유가 좀 있으면 제안해주거나 코너케이스 봐줄수도 있는데 철저히 케바케라
풀리퀘 CI통과하면 코드보고 코멘트로 수정할거 있으면 알려줌
그런데, 코드에 대하여 리뷰를 하는데, 더 나은 방향으로 유도할 수 있는건지 궁금.
비꼬고 부정적인거 아니고, 안해봤음 ㅜㅜ
내 경험상 보통은 더 나은 방향으로 유도하더라 특정 라인에 무슨 의도로 짰는지 물어보는 경우가 많았음 이슈가 열려있으면 설명하기 편해짐
오... 그렇군
기존에 플젝하는 사람들도 전반적인 문맥을 이해하고 있으니까 하긴하더라
직접 관련부분 테스트해보거나 잘아는 사람 멘션해서 확인해보라는 경우도 있고
글쿤
같은 sw를 만들고 있는데 왜 리뷰를 못 받는 단 건지 모르겠네. 그리고 단순한 컨벤션이나 효율적인 코딩 팁만 받아도 충분히 도움되는 건데. - dc App
해외에 PR리뷰 돈받고 해주는 업체도 있던데
시간당 50만원정도였나