2021-02-24(Wed)

항목	내용
학습 날짜	2021-02-24(수)
학습 시간	11:00~24:00
학습 범위 및 주제	Rails Time
학습 목표	Tournament 인덱스 페이지를 완성한다.
동료 학습 방법	eunhkim님과 페어코딩을 진행했다.

상세 학습 내용

문제)

지금은 각 모듈별로 완성시킨 후 develop 브랜치에 한번에 PR을 하는 구조인데, 각 모듈의 양이 상당하다보니 아래 문제들이 있었다.

1. PR을 리뷰하는데 시간이 너무 오래 걸린다. 때문에 PR 리뷰 의무가 있는 팀원들이 원래 자신의 모듈로부터 떨어져있어야하는 시간이 길어지고 이는 각 모듈 개발 속도를 떨어뜨린다. PR 리뷰를 받는 팀원들도 PR이 너무 길어지니 해당 브랜치에서 다음 기능을 추가하는데 어느정도 제약이 있고, 결국 병목으로 작용한다.

2. PR 리뷰에서 지적 사항을 반영하는데 시간이 많이 걸린다. 특히 구조에 대한 지적을 받으면 기존에 작성한 구조를 갈아 엎어야하는 상황이 생기는데, 이미 기존 구조 위에서 구현된 내용이 많으면 공수가 많아진다.

image

무려 229개의 컨벌세이션..

해결)

Optimal pull request size 에 대해서 팀 차원에서 대화가 있었다. 결과적으로 PR 단위를 완결성있게 가져가되, 300줄 내외의 양이 유지되도록 '작게' 가져가자는 것에 다들 동의했다.

지금 우리가 선언해둔 datetime 데이터에서 date만 비교하거나 time만 비교하는 방법을 학습했다.

https://stackoverflow.com/questions/14329198/comparing-times-only-without-dates

https://stackoverflow.com/questions/13257344/can-you-have-a-time-without-a-date-in-rails https://stackoverflow.com/questions/12181444/ruby-combine-date-and-time-objects-into-a-datetime

학습 내용에 대한 개인적인 총평

Tournament index 메서드는 완성했다. 이제 자정마다 Tournament를 생성하는 로직을 확정해야겠지..

eunhkim님이 PR 단위에 대한 문제와 해결책을 제시해주셔서 영감을 받았다. 분명 문제가 있는 상태였는데 타성에 젖어 문제를 인지하지 못하고, 그냥 받아들이고 말았다는 사실에 소름 돋았다. 정확히는 문제를 느꼈음에도 해결대상으로 여기지 않았던 것이 문제였다. 내가 지향하는 방향의 성장을 다시 한번 확인할 수 있었다.

다음 학습 계획

• Tournament 오퍼레이션 기능 완성