[TIL] 2022-10-04

민석·2022년 10월 4일
0

TIL

목록 보기
32/87

Fact (사실)

코드숨 개선 프로젝트

Feeling (느낌)

결국 오늘 아침까지도 axios 모듈만든 테스트 코드를 작성하지못했습니다
소득도없이 시간만 끄니 일단은 허락을 맡고 코드부터 push를 했습니다
axios module 작업은 기존에 하던 작업과 연관이없는데 작업중이던 pr에 올려서
다시 새로 올려달라는 피드백을 받았습니다.

제가 main에서 작업후 pr을 날려 스타트를 잘못끊어 방법을 잘모르겠더라구요..
다행히 동료분들중 그런현상을 겪으신분이 문서화를 해놓으셔서 많은 시간을 잡아먹지않았습니다

axios module 피드백을 받았는데 then catch 이런식으로 요즘엔 사용하지않는다고하시고
제가 만든 axios module에선 header에 무조건 토큰만 넣어서 자동화하게끔했는데
header로 무조건 토큰만 보내진않는다 범용성이 떨어진다라는 피드백을 받아 일단 기간을 맞춰야하니 이부분을 pass 하기로 했습니다

한 pr에서 작업을 너무많이해 방대해지는 바람에 코드리뷰하기 힘든환경이되고 나중에 충돌날 염려가있기때문에 pr을 잘게 나눠서 해야한다는 피드백을 받고 실행했습니다
처음에 다 옮길생각을 하니 멘탈이 나가있었는데 pr을 쪼갠뒤 첫 피드백을 받았을때 이정도 사이즈로 하면 되겠다는 말을 듣고 멘탈이 좀 회복되었습니다

2번째 pr에서 양조절을 실패한거같습니다
2가지 기능을 한꺼번에 pr했었는데 저는 길거라 생각안했는데 생각보다 코드가 길어지더라구요
앞으로는 1가지 기능을 담당해서 pr을 해야겠습니다

프로깃 책도 읽으려했는데 어느덧 밤이늦어 내일로 미뤄야겠습니다

Finding (교훈)

axios 테스트코드때문에 2일은 고민했던거같은데 이기간동안엔 정말 진도가 안나가서 너무 힘들었습니다.
2일동안 고생했어도 pass하게 되니 너무 다행이다 싶더라구요 맞서서 깨는게 멋있긴하지만..

피드백텀이 너무 느리다라는 피드백을 받았었는데 이제 어느정도로 쪼개서 pr을 할지 감이 조금 오는거같습니다 다음 스프린트땐 더 잘할수있을거같아요

아직 눈에 띄게끔 성장했다는 느낌을 못받고있는상태지만
지금 오는 스트레스와 불편한 감정들이 느껴지는게보여 확실히 성장하고있구나 라는 생각이듭니다
화이팅..!

profile
안녕하세요 프론트엔드 개발자 양민석입니다.

0개의 댓글