내일배움캠프 D+88: 0714

🎶 최종 프로젝트 15일차

🤔 무한한 깃의 세계

부끄럽게도 오늘에서야 비로소 틀이 잡힌 느낌이 들었다. 팀원 한 분을 제외하고는 모두 나처럼 같은 브랜치에서 계속해서 작업을 하고 있었는데, 오늘 내가 한 pr에 커밋이 왕창 들어가서 개선해야할 필요성을 느꼈다. 튜터님께 조언을 구하니 애초에 pr은 커밋 단위로 짧게 합치고 해당 브랜치를 삭제하는 것까지가 하나의 과정이라고 하셨다. 그래서 우리는 기존의 방식을 수정하기로 했다.
일단 이슈에 할 일을 등록하고 브랜치를 그에 맞게 생성한 후 해당 이슈를 해결하면 바로 pr하여 하나의 pr에 관련된 내용만 들어갈 수 있도록 했다. 항상 코드 리뷰에 대해 어려움이 있었는데 이렇게 간결하게 진행하니 팀원들도 리뷰를 하기 편해져서 댓글도 잔뜩 늘었다!
커밋할 때는 항상 태그가 헷갈려서 팀원분들이랑 상의하고 큰 묶음으로 주된 태그를 정했다. 그나마 편해질 것 같다.
덧붙여 pr을 할 때 커밋 과정에서 머지 커밋이 생기는 팀원분들이 있기에 rebase 제안을 드렸다. 아무래도 그런 머지 커밋이 끼어있으면 보기 불편하기에 변경하기로 했다.

😱 해결이 시급했던 버그

간헐적으로 자동 로그인 처리가 되거나 GET 요청이 토큰 관련으로 기능하지 못하는 경우를 해결하려고 튜터님께 조언을 구했다. GET 요청만 인가를 제외하거나 비동기 처리되는 걸 해결하면 된다고 하셨다. 생각해 보니까 개발할 때 GET을 제외해야되나 싶었는데 DB저장 과정이 없었기에 큰 문제가 없었다. 그래서 그 심각성을 고려하지 못했던 것 같다... 반성하자.

😖 끝이 없는 리팩토링

내가 맡은 기능이 생각보다 너무 많았다. 한 가지 기능 이상을 한 수준인데, 그걸 하나의 패키지에 다 넣어둬서 아주... 좋지 못한 피드백을 받았었다. 일단 팀원분들이 알아온 대로 공통적인 부분만 분리했었는데, 리팩토링을 하다보니 아예 새로운 패키지로 분리가 필요하다고 생각했다. 급한대로 간단한 분리는 했는데 추가적인 리팩토링이나 dto추가는 필요할 것 같다.

일주일이 더 주어졌다. 그래도 내일 더욱 의미있는 평가를 위해서, 내일 오전에도 최대한 열심히 이슈들을 해결해야겠다.

✅ 응답 DTO
✅ 프로필 업데이트 오류 수정
⬜ 리팩토링
⬜ 버그 해결
⬜ 예외 처리