RIEN😚
이상한 나라의 개발자
RIEN😚
전체 방문자
오늘
어제
  • 분류 전체보기 (125)
    • Algorithm (68)
      • 알고리즘 (0)
      • Baekjoon (8)
      • 프로그래머스 (55)
      • HackerRank (5)
    • Android (30)
      • Project (1)
      • Error (2)
      • Studio (1)
      • Android (26)
    • Kotlin (6)
    • CS (4)
      • 네트워크 (2)
      • 데이터베이스 (2)
    • Front End (5)
      • React (1)
      • VUE (3)
      • Project (0)
      • 기타 (1)
    • 기록 (11)
      • 회고록 (6)
      • TIL (5)

블로그 메뉴

  • Github🔥
  • 포트폴리오🌹

공지사항

인기 글

티스토리

250x250
반응형
hELLO · Designed By 정상우.
RIEN😚

이상한 나라의 개발자

기록/TIL

[TIL] 2022.07.15 금요일

2022. 7. 15. 22:07
728x90
반응형

우테캠 팀 프로젝트의 첫 주가 끝났습니다.

다른 팀들이 어떻게 프로젝트를 진행하고 있는지 들을 수 있는 시간을 가지게 되었어요.🤗

 

정말 다들 대단하시더라고요!

이번 중간발표에서 크게 3가지를 배울 수 있었어요. 😊

 

📌 Android Convention 정보
📌 Issue와 PR, 그리고 코드리뷰
📌 요구사항과 일정 정리 문서: WBS

 

1. Android Convention

저희 팀도 물론 프로젝트를 시작하기 전에 Git 규칙과 Code Convention에 대해 정리를 해두었지만,

다른 팀분들이 정리하신 문서를 보니, 저희 팀에서 생각하지 않았던 사항들이 많이 포함되어 있는 것을 보게 되었어요.😌

 

그래서 그 분들에게 양해를 구하고 문서를 훔쳐오게 되었습니다.😆

🌹 처음에는 각 Convention에 맞게 구현하기는 쉽지 않겠지만, 차근차근 리팩토링을 해가면서 연습해볼 예정이예요.👏🏻

 

2. Issue와 PR, 그리고 코드리뷰

저는 화면 하나를 구현해야 할 때, Issue 하나를 두고 하나의 브랜치에서 작업을 하고 있었어요.

 

👇🏻 하나의 이슈에 정리해둔 세부 구현사항들

그리고 Issue와 관련된 PR에는 세부 구현사항들과 대응되는 무수한 커밋들이 있었습니다.ㅎㅎ...

그러다 보니 하나의 화면을 다 구현할 때까지 한 개의 PR도 요청하지 못하는 상황에 빠지게 되었어요.😭

 

PR을 못 날리니, 코드 리뷰도 그저 구두로만 진행이 되었었습니다.

🌹 지금 돌아와 생각하니, 정말 후회스러운 상황이네요.

 

하지만 팀 발표 중 다른 팀원분들의 PR을 보니, 세부 구현사항 별로 Issue와 PR을 만들고

코드리뷰를 통해 많은 의견을 나누시고 계시는걸 보게 되었습니다. 😳😳

(또한 페어 프로그래밍을 통한 협업도 해보시고 계시더라고요.)

 

그래서 저희 팀은 발표가 끝난 뒤에 다시 한번 회의를 진행하게 되었습니다.

👇🏻 추가된 PR과 리뷰 규칙 + 페어프로그래밍 일정

남은 프로젝트 일정이 짧은 만큼 작은 단위로 Issue를 만들고, 해당 Issue별로 PR를 만들기로 하였습니다.

PR을 날리게 되면 바로바로 공유할 수 있도록 하고,

공유를 받은 2시간반 이내에 코드 리뷰를 작성해주기로 했어요.🔥🔥

🌹 이를 통해 빠른 템포로 개발을 진행하면서도 의견을 교환할 수 있는 기회를 최대한 가지는 것을 목표로 하고 있어요.🤗

 

👇🏻 그래서 바로 팀원분이 금일까지 작업한 내용을 PR해주셔서 빠르게 코드리뷰를 남겨드렸습니다.👍🏻

 

또한, 지금까지 너무 각자 역할을 완전히 분할하여 구현하고 있어, 협업에서의 경험을 많이 가져가지 못하고 있다는

지적을 받게 되어, 남은 화면 하나를 함께 구현해보기로 날짜와 방법에 대해서 이야기를 나눠보았습니다.

 

3. 요구사항과 일정 문서 정리: WBS

👇🏻 현재 저희 일정 계획표

처음에는 이 정도의 일정 계획을 바탕으로 kanban 보드를 작성하면서 일정 관리를 하면 충분하다고 생각했었는데,

다른 팀들 중 한 팀이 WBS 문서를 작성하여 요구사항과 일정을 한번에 관리하는 것을 보여주었습니다.

(WBS라는 것도 처음 들어봤어요.😭)

 

정말 깔끔하게 잘 정리되어있더라고요.

 

다음 프로젝트를 시작할 때는 이 문서를 적극 참고해서 작성해보고자 합니다.

 

4. 추가  - 마스터 코드리뷰

- 코드의 일관성을 유지하면서 프로그래밍 하기!

- 객체가 어떤 역할을 하는지 확실히 정하고, 관련된 코드만 관리하도록 하기!

- 특정 라이브러리를 사용할 때는 항상 이유(Why!)를 생각하기

 


[마치며]

의외로 지금 우테캠의 200% 만족 중입니다.😆

 

일단 대단하신 분들과 코드리뷰를 하면서 여러 새로운 지식을 배우고 잘못된 지식을 바로잡는 과정이 너무나도 재미있고,

다른 분들의 코드를 읽으면서 코드리뷰를 남겨드리는 것도 너무나 재미있네용😍

(문서 작성은 덤!ㅎㅎ)

프로젝트 진행만큼은 처음에는 짧은 기간에 의해 약간 힘든점이 있었지만, 완전 즐기고 있는 상황입니다.ㅋㅋ

 

앞으로는 팀원분과 코드리뷰를 적극적으로 하면서 정보를 교환하자고 의견을 모았습니다.🔥🔥

이제 팀원분이 작성해주신 코드리뷰에 답변을 해드리고 리팩토링할 부분을 정리해야겠네요..

 

내일도 화이팅!!

반응형

'기록 > TIL' 카테고리의 다른 글

[하루네컷] ContentResolver에서 최신순 정렬을 위해 삽질한 결과  (0) 2023.04.27
[하루네컷] UiEvent에 StateFlow 대신 SharedFlow를 사용해보다.  (0) 2023.04.20
[TIL&일일회고] 2022.08.01 월요일  (0) 2022.08.02
[TIL] 2022.07.14 목요일  (0) 2022.07.14
    '기록/TIL' 카테고리의 다른 글
    • [하루네컷] ContentResolver에서 최신순 정렬을 위해 삽질한 결과
    • [하루네컷] UiEvent에 StateFlow 대신 SharedFlow를 사용해보다.
    • [TIL&일일회고] 2022.08.01 월요일
    • [TIL] 2022.07.14 목요일
    RIEN😚
    RIEN😚
    안드로이드 / 코틀린 독학으로 취업하자!

    티스토리툴바