개발문화 질문입니다.

Question

다들 개발하실때 깃헙을 많이 사용하실겁니다. 혹시 PR(Pull Request)를 적극적으로 활용하시나요??하신다면 소스리뷰를 요청하는 단위는 어떻게 되나요??지금 리드하는분이 소스리뷰를 PR단위로 하긴하는데 이게 시간을 너무 많이 잡아먹습니다. 일단 제대로 돌아가고 속도에 이슈가 없다면 저는 OK라고 생각합니다.하지만 이상적인것에 빠져있는건 아닐지 의심되네요....리팩토링도 좋고 코드 정리도 좋지만 일단 일정이 우선되야 하는게 맞지 않나 싶습니다. 다른사람이올린 PR때문에 내가 개발하던것도 놓고 봐줘야 하고...사실 보는게 라인단위로 보는것도 아니고 그냥 브런치 실행시켜보고 해당기능 구동에 이상없는지만 확인해야하는데 ...이건뭐...라인별로 테클이 들어오니... 효율도 떨어지고 팀원들 사기도 떨어집니다...코드리뷰를 주기적으로 하시는지...다른사람의 코드를 테스트한다면 개발자 입장에서 어디까지 해야하는것인지가 의문입니다.

Mux · Accepted Answer

1. PR을 진행하기전 코드 규칙(Convention)을 정하고 진행하자.2. 코드를 작성할 때, 본인이 작성한 코드는 전부 본인이 이해를 하고 있어야 한다. 3. 작성한 코드에 대하여 리뷰를 진행하는 팀원들을 위해 코드에 대한 설명을 달자.4. 너무 많은 코드를 한번에 PR하려고 하지마라. 5. PR을 진행하기 전, 반드시 작성한 코드에 대하여 테스트를 진행하라.6. PR을 작성할 때, 작성 규칙을 정하자. 예를 들어 리팩토링이면 [Refactor] 라든지, 특정 문구를 남기는 것이 좋다.

ACACIAN · Answer

제가 선호하는 개발문화네요!

Mux · Answer

위에 적은 내용 말고도 좋게 사용할 수 있는 방법은 많습니다. 다만 대표적으로 사용하는 마인드를 정리해 보았으니 도움되셨으면 좋겠습니다.

최현일 · Answer

스마트 기기. 안전하게 사용합시다. 좋은 습관을 만들어요. Pull Request를 사용한다고 하셨는데...개발 할때에, 처음에, git clone으로 복사하구요. 다음부터는, git pull로 받구요. 그리고, 스프링부트의 경우, ./gradlew build, 코딩하고... git push 이렇게만 해주어도 되죠. 그런데.. 코딩을 한다는 말이 맞기는 한데... 집에서 사용하니까.. git을 private로 코딩해서 올리고...(git push). 서버. 호스팅을 켠 뒤에, git pull로 받고, ./graldew build. war파일 빌드하고... screen만들어서, java로 서버를 켜기. 이렇게 되더군요.오타인가.. ./gradlew 이렇게 적어야 되는 것 같았군요. 여러명에서 개발을 할 경우에는, 개발을 2개로 두고... 테스트용, 서비스용(서비스 전의 모양). 이렇게 만들고, git clone을 테스트용으로 받아서...이때가 pull. 그리고, private로. 코딩하고 push하구요. 그 모양을 보고, 서비스용에 옮겨 적기. 이렇게도 되겠죠. 이렇게 개발하는 환경이라면, 옮겨적기 좋게, 따로 서류로 만들어 준다면... 아니면, 이 서비스가 프로토타입으로 좋다. 그러면, 그대로 한번 더 쳐도 되겠구요. 그래서, git코드를 가지고 있을 수 밖에 없다. 이렇게 생각해도 될 것 같아요. private를 사용하니까 그렇죠. 개인정보의 경우는 SQL인데... 있을 수 없겠죠. 쉽게 만들 수 있는 부분도 있겠구요. 젠킨스. 이런 것도 있다고 하구요. CI이런 부분도 있다고 하더라구요. 이 책 좋은 것 같습니다. 공부해보세요. '스프링부트와 AWS로 혼자 구현하는 웹 서비스' 추천해보네요. CI부분. 사용방법도 있습니다.또, 빌드방법도 있구요. 개인적으로, AWS는 비추천합니다. 메일보내고... 소통이 잘안되는 부분. 인스턴스가 잘 안지워지데요. 모르고 2개 만들었군요. EC2서버. 작은것. 1개까지는 무료. 구름IDE추천해보네요. 그래도 공부도 많이 되었구요. 공부했다는 수업료. 이렇게도 생각해보고 있는 군요. 공부해보세요~. 좋은 개발되세요... 또, 교회. 다녀보세요. 기독교. 전화도 해보세요. 하고 싶은 말이라서 적게 되네요. 좋은 하루되세요. 저의 댓글, 봐 주셔서 감사합니다.

개봉박살 · Answer

Mux좋은 팁 감사합니다.

잡초 · Answer

코드 컨벤션은 spotless 같은거 git hook 으로 강제 할수는 있습니다.커밋 자체가 안되요 spotless 실행을 안하면 그러면 코드 컨벤션은 지켜 지는데, 문제는 저는 오히려 가독성이 더 떨어 집니다.;;위에 문화를 겪어 봤는데, 적응 되시면 속도가 좀 나오긴 하실 거에요 .근데 문제는 개발 단계부터 저렇게 pr 하시고 일정에 문제가 있다고 하시면 안하는게 맞습니다.지금은 개발 단계라 pr을 하지 않고, 운영 으로 전환 해서 배포 하기전에 pr 합니다.지라 티켓 따고 해당 버젼으로 브랜치 따고 pr 올리고 확인 받고 배포

개발문화 질문입니다.

6개의 답변

#pullrequest 목록