Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

이전 프로젝트 피드백 요약 #1

Open
Hyune-c opened this issue Mar 27, 2020 · 2 comments
Open

이전 프로젝트 피드백 요약 #1

Hyune-c opened this issue Mar 27, 2020 · 2 comments

Comments

@Hyune-c
Copy link
Contributor

Hyune-c commented Mar 27, 2020

Master 피드백

Honux

readme

  • 리드미 페이지는 대문이니까 조금 더 신경써서 깔끔하게 만들면 좋겠다.
  • 리드미 페이지는 대문이니까 조금 더 신경써서 깔끔하게 만들면 좋겠다.중간에 브랜치 테이블에 local, origin으로 구분해 놨는데 구분의 의미를 모르겠다. origin은 git에서 사용하는 별칭이라 어색하고, 테이블의 쌍도 맞지 않다. 그냥 블릿으로 적어놓는 게 좋았을 듯
  • 리드미 페이지는 대문이니까 조금 더 신경써서 깔끔하게 만들면 좋겠다.API 문서의 포스트맨을 한글로 적어 놓은 것도 매우 어색

프로젝트 동작

  • 회원가입은 엄청 잘 만들었다.
  • 아이디 중복체크를 포커스를 벗어날때 하는데 그것보다는 한글자 입력할 때마다 하는게 더 일반적인 듯. 다 입력하고 다음칸 넘어갔는데 경고 문구가 나오니까 다시 돌아가야 해서 어색
  • 관심사에 같은 관심사를 3 개 입력해도 정상 동작(?) 코딩, 코딩, 코딩

프로젝트 관리

  • 브랜치는 적절하게 잘 사용하는 듯
  • 간혹 이슈에 제목만 짧게 있는 경우가 있는데 가능하면 한 두줄이라도 설명을 추가하면 좋을 것 같습니다.

Crong

  • 409상태로 응답하는 것 좋아보이네요.
  • 커밋메시지 간결하고 가이드 좋음. 한글로 적는것도.
  • 스크럼은 관리이슈는 아니라서 wiki를 활용하는 건어떨가 생각됩니다.
  • PR을 기능별로 잘 날리고 있어서 좋아요.
  • task 들의 subtask 관리까지 자세하게 해서 좋네요.귀찮은 일일텐데.
  • 스크럼내용이나 기타 기술정보는 github의 wiki를사용하는 것도 좋죠.
  • 커밋할때 대문자로 시작하는 거 보면 좀어색해보여요. 대문자,소문자 막 섞여 있음 ㅎ
  • 브랜치를 잘 나눠서 진행하시는거 같아서 좋음
  • 진행사항 관리가 projects 에서 잘 되고 있네요.금요일이라 그런지 TODO도 몇개 없군요
  • 배포준비를 잘 하고 있겠지만, 다음주에는 좀더미리미리 준비 하도록 해보세요.
  • issue에서 스크럼을 하면서 서로 이야기하는 모습아주 좋네요. 칭찬 격려 이런게 제일 중요해요.

개선해야 된다고 생각하는 점

아래 내용은 합의되지 않은 저의 의견이며, 월요일 전까지 지속적으로 추가될 수 있습니다.

README

  • 리드미 페이지를 어떻게 깔끔하게 할 것인가?
  • Branch 항목에 대한 정리.
  • README 보다는 WIKI 에 있어야 할 항목에 대한 분리.

Issue

  • 크롱의 긍정적인 피드백 처럼 Task 에 대한 설명을 자세하게 기록하는 게 좋을 것 같습니다.
    • 저 같은 경우는 처음의 Issue 에는 할 기능을 적어놓고, PR 을 보내는 시점에는 실제로 구현한 세부 내용을 추가로 적곤 합니다.

Commit

  • 커밋을 시작하는 문자가 섞여 있다고 합니다.
    • 저는 대문자 시작에 한표!
  • Footer 에 이슈 번호 (# 이용) 를 담으로서 이슈와 PR, 커밋간에 관계를 가져갈 수 있습니다. 활용하면 좋겠습니다!

Deploy

  • 다음 프로젝트는 AWS 를 기반으로 작업합니다. 주말간 AWS 를 올려보고 최소한 BE 끼리는 접근 권한을 공유할 수 있는 방법을 고민해보겠습니다.

문서 정리

  • 문서 정리의 담당자 지정이 필요할듯 합니다.
    • 피드백 중 문서에 관련된 내용은 스크럼의 위키화, POSTMAN API 의 체계화 였습니다.
    • 프로젝트가 진행됨에 따라 문서 관리의 범위와 난이도는 커지지만 그 담당이 모호합니다.
    • ex) 요구사항분석+산출물: dan
      , postman : diane, taek
      , readme+scrum+wiki : delma

프로젝트 구조

  • 이번 주는 프로젝트 Root 를 Repository 와 동일하게 갈 수 있습니다.
    • 가능하면 나누지 말고 가서 배포에 subtree 를 쓰는 불편함이 없었으면 좋겠습니다.
@delmaSong
Copy link
Member

리드미 페이지

  • 멤버
  • 기획서 링크
  • 데일리 스크럼 위키 링크
  • api 문서 위키 링크
    이정도만 넣는 건 어떨까요?
    상세하게 필요한 내용은 위키에 작성하고 링크 연결하는 식으로

커밋

이슈 번호로 관리하는게 아직 익숙하지가 않지만 단이 작성한 내용 둘다 적극 추진하면 좋을 것 같아요!

문서정리

담당자 정하는 것도 아주 좋습니다!
저대로 정해져도 좋을 것 같아요

===================
단이 이전 프젝에서 보완할 것들 잘 정리해주셨네요!
앞에서 좋았던 점은 그대로 가져가고 나머지 멤버들과 보완사항 마저 정해서
이번 일주일도 잘 해봐요 🌷

@moonelysian
Copy link
Contributor

일단 저희 다 브랜치 전략이나 전반적으로 커밋 전략은 잘 하고 있다는 피드백을 받았으니까 저번처럼하되 add feature 이런것만 소문자로 바꾸면 될꺼 같아요!

브랜치

  • be(be의 마스터 개념)
  • be/dev
  • be/feature/..
  • be/ 기타등등

be를 예시로 들면 기능 완성 후
be/dev로 pr 보내고 확인하고 머지 dev가 어느정도 쌓이면 be로 머지 그리고 배포 이런식으로 돌아가면 될꺼 같아요..!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants