TIL]2022/09/29

아침에 상쾌하게 일어났습니다.

아침에 일어나 공부방에 갈 준비를 하며

오늘 해야 할 일을 정리했고

가장 먼저 해결해야 할 문제가 무엇인지

계속 생각했습니다.

우선은 마이그레이션에서 발생한 이슈를 빠른 시간 안에 해결하고

만약 무언가 발목을 잡는다면 바로 버전을 롤백하는 것을 생각했습니다..!

 

공부방에 도착 후, 데일리 스크럼 미팅을 진행했습니다.

마이그레이션 이슈에 대한 부분에 대한 이야기가 나왔고

오전 내에 해결하지 못한다면 버전을 롤백하기로 결정이 났고

코드 리뷰에 대한 컨벤션을 정하는 시간을 가졌습니다.

 

pr은 origin에서 브랜치를 판 뒤 코드 숨 upstream 메인으로 pr 합니다.
작업을 모두 완료한 뒤 pr을 요청하는 게 아닌 최소 요구사항을 만족시킨다면
pr을 올리고 디스코드에 공유합니다.
pr을 open 한 사람을 제외한 모든 사람들은 급한 일정이 있는 게 아닌 이상 코드 리뷰에 참여합니다.
모든 프론트팀이 리뷰를 완료하면 pull request를 연 사람이 merge 합니다.

위와 같이 Pull Request를 보내기로 하였고

코드 리뷰에 대한 가이드 문서의 방식대로 코드를 리뷰하기로 했다.

자료를 읽는 것은 정말 중요한 것 같습니다.

위 문서에 있는 내용을 요약하자면 다음과 같습니다.

---

 

에지 케이스에 대해 생각합니다.

사용자처럼 생각하려고 노력합니다.

코드를 읽는 것만으로 보이는 버그가 없는지 확인합니다.

집중된 작업을 하고 있지 않다면 코드 리뷰가 들어온 직후에 코드 검토를 수행해야 합니다.

오버 엔지니어링을 주의합니다.

네이밍을 신경 씁시다.

코드 스타일을 따르고 코드가 일관성 있는지 확인합시다.

---

 

코드 리뷰를 어떻게 해야 할지, 감조차 오지 않았던 저에게

코드 리뷰는 이런 식으로 진행해야 합니다라는

가이드 문서를 보면서 확실히 느낀 것은..

자료를 읽는 것은 정말 중요한 것 같습니다.

 

데일리 스크럼 미팅이 끝난 후, 마이그레이션 작업을 진행했고

정상적으로 동작하는 것을 확인하고 롤백을 하지 않아도 되는 것에

기쁨을 느끼면서 코드를 pull request를 하기 전 코드 점검을 진행했습니다.

 

점심을 먹은 후 정리한 코드를 pull request를 하고

다시 미팅을 진행했는데

미팅 내용은 코드 숨 공부방 좌석 예약이 정말로 필요한 서비스 일지

수요조사를 하기 위해 노션으로 간단히 수요조사를 위한 사이트를 만들자는 내용이었습니다.

이번에 수요조사를 하는 인원은 4명이었고

4명이서 공부방에 와서 얻을 수 있는 장점들을 생각하였습니다.

어떻게 하면 사람들이 글을 읽을까?

한참을 고민한 것 같습니다.

처음에는 공부방의 장점들을 각자 나열하였고 그 후에

어느 정도 장점들이 나열된 것 같아

2명은 장점에 살을 붙이는 작업을 하고

저 포함 2명은 포스터를 만드는 작업을 진행했습니다.

포스터 작업이 완료된 후, 장점에 살을 붙이는 작업을 합류하여

다시 진행하였는데

생각할 것이 많았고 어떻게 하면 사람들에게 이 장점을 어필할 수 있을까라는 고민 끝에

완성을 한 후 설문조사 폼을 제작하였습니다.

이번 조사를 통해서 프로젝트가 정말 필요한 지, 조사할 뿐 아니라

프로젝트의 방향 또한 정할 수 있는 것 같아서

수요조사를 하는 것은 중요하다는 것을 깨달았고

프로젝트를 진행할 때, 무작정 프로젝트를 진행하는 것이 아닌

실제 예비 유저들에게 한번 설문을 하고

실제 예비 유저들을 설득하기 위해 어떻게 장점을 어필해야 하는가를 생각해야 한다는 것을 배웠고

좋은 경험이었던 것 같습니다.

 

집에 도착 후, NextJS 마이그레이션을 하며 생긴 이슈 문서화를 수정하는 작업을 진행하였는데

문서화는 생각보다 어려운 것 같습니다.

문서화를 하는 이유는 사람들이 이 글을 읽고 같은 장애물을 만났을 때, 대처법을 알려주며

가독성 또한 좋아야 하는데 많은 부분을 놓치고 있었던 것 같아서

수정을 하며 많은 반성을 하게 된 것 같습니다.

아직은 문서화가 익숙하지 않지만 문서화를 하는 부분도 많이 연습을 해서

문서화가 익숙해져야 할 것 같습니다.

문서화 수정 작업을 마친 후,

코드를 분석하며 emotion에 대한 부분에 이해가 되지 않는 부분이 있어

emotion 공식문서를 읽으며 정리하는 시간을 가졌습니다.

당분간은 emotion 공식문서를 읽으며 정리하는 시간을 가져야 할 것 같습니다.