Skip to content

2주차 회고

MinHK4 edited this page Nov 26, 2022 · 1 revision

팀 KPT 회고

1. Keep

  • 시간관리 타임테이블대로 나누고 진행 & 공유한 점이 좋았다.
  • 팀 내부 분위기 이번주처럼 부드럽게 유지합시다!
  • PR 크기가 적당했음 → 1일 1PR

2. Problem

  • PR한 후 리뷰 언제할 지 애매했던 부분…
  • Merge가 되어야만 올릴 수 있는 PR이 존재하는데, merge가 되지 않아서 확인 못하는 문제가 발생함 → 작업하는데는 영향이 없음. feature 브랜치에서 pull 받아서 구현하면 되지만 이후 날린 PR을 먼저 merge 하면 안 됨. (Merge의 선후 관계 존재) → 이 점을 인지하고만 있으면 PR을 올릴 수는 있음.

3. Try

  • 작업이 끝나면 PR은 꼭 올리자!!

  • PR 리뷰 시 1시간 내 리뷰해야한다는 그라운드룰을 지키자

    → 가능할까..?

  • PR 날린 후 slack에 링크 올리기, 다음 주부터 분업해서 작업한다면 공유의 필요성이 더 커질 듯

    → 꼭 슬랙에 올리지 않더라도 확인하는 데 어려움은 없을 듯함

  • Feature 구현 끝난 후 PR이라면 그라운드룰 대로 진행하고, 아닐 경우라면 PR 날리고(1일 1PR) 코드 리뷰는 가능한 시간에 자유롭게 하자

    → PR은 많을수록 좋지만 PR 자체는 기능 구현이 끝난 후 날리는 것이므로, 구현이 완료되지 않은 시점에 날리면 더 헷갈릴 수 있음.

    → 따라서 작업 단위를 작게 나눠서 PR 크기는 작게 가져가되, 기능 구현이 끝난 후 날리자

    → 왔다갔다하면서 틈틈이 중간 과정 리뷰해줄 수 있으므로 더 좋지 않을까!

    → 기능 구현이 끝나고 merge해야 될 때 무조건 리뷰를 해야한다는 그라운드룰이 있었음. 이 때 최종 리뷰가 이뤄졌는지 구분할 수는 있어야 함.

    → 기능 구현이 끝나야 코드 이해가 편함… 중간 상태에서 코드를 잘 이해할 수 있을까?

  • PR 방식 논의

    1. 작업 끝난 후 PR 올리기 vs 2. PR을 먼저 올려놓고 작업 (핵심은 이동하면서 다른 사람 코드 볼 수 있게)
  • 리뷰는 누가?

    페어프로그래밍(협업)을 했다면 꼭 리뷰가 필요할까? → X, 같이 한 번 검토하고 merge

    페어 내에서 분업을 했다면 리뷰 필요 → 짝꿍끼리 가능함

  • 리뷰 기한

    • 코어 타임일 경우: 1시간 이내 but 급한 경우 직접 요청
    • 코어 타임 이후: 다음날 오후 2시 이전
  • 리뷰 후 반드시 수정해야 하는 오타는 수정하고 merge (구분 필요)

  • 데일리스크럼 이전까지 되도록이면 상대 페어 코드 보고 오고, 공유 필요한 사항을 데일리스크럼 때 공유

Conclusion

  • 작업이 종료되면 PR을 올리자
  • 올린 PR은 페어프로그래밍으로 진행한 경우 → 리뷰 필요 X, 분업인 경우 → 리뷰 필요
  • 리뷰 기한은 코어타임일 경우 1시간 이내, 코어타임 이후인 경우 다음날 오후 2시 이전
  • 상대 페어 코드는 틈틈이 읽어오고 공유 필요한 사항만 데일리스크럼 때 공유

다음 주에 논의해야 할 사항

  • BE repository layer을 둬야 할 지?

주말에 해올 것

❗ : 필수

  • 현재 구조에 익숙해지기
  • Next.js 더 공부해오기: 버그 ..
  • ❗ 에디터 라이브러리 탐색해보기
    • Draft
    • TOAST UI 등등
  • ❗ UI 작업
  • ❗ API 명세 example 추가
    • Auth + User: 현기
    • Book: 다현
    • Article: 민형
  • CI/CD (도현)

개인 회고

김도현

사실 (Fact):

느낌 (Feeling):

교훈 (Finding):

향후 행동 (Future Action):

김민형

사실 (Fact): 로그인 구현 시작 및 완료!

느낌 (Feeling): 기획보다는 구현에 들어오니 본격적으로 시작하는 느낌이 들었다 그대신 코드의 감이 녹슨 것 같은 불안함이…

교훈 (Finding): 나의 의견이 커지는 것보다 팀의 의견이 정확해지는 것에 초점을 맞춰야 한다

향후 행동 (Future Action): 개발을 시작할 거니까 조금 더 준비가 잘 되어있어야 할 것 같다..!! NextJS에 대해서 더 친숙해져야 한다!!

박현기

사실 (Fact): 첫 팀프로젝트 개발 시작!

느낌 (Feeling): 기획에서 많이 고생한 보람이 있었다! 1주차보다 의견을 모으고 결정하는데 시간을 단축할 수 있었다.

교훈 (Finding): 팀프로젝트에서는 의견을 모으고, 효율적으로 시간을 분배하는 것이 중요하다!(하지만 어렵다)

향후 행동 (Future Action): 이제 진짜 개발 시작, 열심히하자

정다현

사실 (Fact): 드디어 VSCode를 켰다!

느낌 (Feeling): 구현을 하다보니 저번주보다는 덜 힘든 것 같다. 목표했던 바를 다 이루진 못해서 아쉽지만, 진행 방식을 조율하기 위해 소통했던 과정이 좋았다.

교훈 (Finding):

향후 행동 (Future Action):

Clone this wiki locally