본문 바로가기
TIL/React

NEXTSTEP React Study : Week 2 - PR approve 전 확인, 코드 리뷰 팁

by VANAV 2023. 4. 18.

코드리뷰

나보다 잘 하는 사람의 코드는 어떻게 리뷰해야 할까?

어떻게 코멘트를 남겨야 도움을 드릴 수 있을까?

 

 

스터디를 하면서 실력이 뛰어난 분의 코드를 보고 도움이 될 리뷰를 남기고 싶었는데 어떻게 남겨야 할 지 막막했다.

그분의 코드를 보고 내 코드를 보니 너무 부끄러웠는데.. 핳하 😂

 

그래서 코드 리뷰와 관련한 다른 분들의 의견을 찾아서 읽어보고,
일단은 코드를 작성하신 분께서 사용하신 스택을 따로 검색하며 공부해 보았다.

실력자분들의 코드를 보면 감탄과 반성만 나오는데 그런 것만 리뷰에 구구절절 적고싶지는 않았기에! 파팅 😭

 

그 중 남겨서 두고두고 읽고 싶은 글들은 여기다 박제...

 

 

구글링하다가 찾아서 정독한 좋은 글

https://hyeon9mak.github.io/code-review-know-how/

 

한번 듣고 평생 써먹는 코드 리뷰 노하우 정리

OKKY 1월 세미나에서 백명석님의 “한번 듣고 평생 써먹는 코드 리뷰 노하우”를 듣고 정리한 내용

hyeon9mak.github.io

 

PR(Pull request)를 승인하기 전에 local에서 테스트 해보기!!

눈으로만 코드를 쫓지 말고 local 환경에서 테스트 해보면서 코드를 봐야 더 상세한 리뷰를 드릴 수 있을 것 같아서

PR 승인 전에 local에 코드를 pull 받아볼 수 있는 방법을 찾아봤다.

 

https://blog.outsider.ne.kr/1204

 

GitHub의 Pull Request를 로컬로 가져오기 :: Outsider's Dev Story

지난달에 [Pull Request를 이용한 개발 흐름을 적용해 보고 나서...](https://blog.outsider.ne.kr/1199)를 통해서 Pull Request를 이용한 개발 흐름에 대해서 공유했다.(기대보다 반응이 훨씬 좋은 글이었다.)...

blog.outsider.ne.kr

https://www.lesstif.com/gitbook/github-pr-pull-request-merge-local-30704954.html

 

github PR(pull request) 을 merge 하지 않고 local 로 받기

아래 예제는 github 프로젝트 URL 이 git@github.com:YOUR_ACCOUNT/PROJECT.git 이므로 실제 URL 로 변경해야 됩니다.

www.lesstif.com

 

이런 것들도 있을까 싶어서 찾아보니 있어서

깃은 정말 다양하고 멋진 기능들이 있구 새삼 나는 지금까지 우물안의 깃무새였구나 하는 반성이 찾아온다
(add-commit-push....) 🦜

 

 

 

스터디의 본격적인 내용 말고도 이런 것들을 공부하면서 성장하기도 할 것 같아 아까워서 글로 백업을 하게 됐다

무언가 알아보고 찾아보고 다른 분들과 소통하며 나는 지금까지 뭣도 모르고 있었음😱을 깨닫는 건 항상 슬프고 괴롭지만 그래도 요즘은 이 시간이 지나가면 더는 괴롭지 않게 된다는데서 위안을 찾는다

 

뭐 git bash가 뭔지도 몰랐고, 커맨드 입력하며 벌벌 떨었던 나날도 있었으니까
그때에 비하면 지금의 나는 깃헙도 써봤고 깃랩도 써봤고 소스트리까지 할 줄 알고 아이구 너무 멋있다 잘하구 있다

최고의 주니어 애기 개발자!! 😤 언젠가는 오늘도 기억나지 않을 정도로 많이 성장해 있을 것이다

 

 

내일도 화이팅

댓글