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

2주차 #5

Open
7 of 9 tasks
yeonhwakim opened this issue Aug 11, 2020 · 4 comments
Open
7 of 9 tasks

2주차 #5

yeonhwakim opened this issue Aug 11, 2020 · 4 comments

Comments

@yeonhwakim
Copy link
Collaborator

yeonhwakim commented Aug 11, 2020

1. 이번주 제한조건 (codesoom 과제 제한조건 참고)

  • 모든 기능은 TDD로 구현합니다. 모든 컴포넌트와 reducer, action에 대한 테스트가 존재해야 합니다.

  • if는 사용할 수 있어도 else 사용하실 수 없습니다. GuardClauses 방법을 사용해주세요.

  • switch는 사용하실 수 없습니다.

  • let은 사용하실 수 없습니다. const만을 사용하여 문제를 해결해주세요.

  • 함수 이름과 변수 이름에 줄임말은 사용하실 수 없습니다. 길더라도 명확하게 써주세요.

  • indent(인덴트, 들여쓰기) depth를 1로 유지해주세요. 예를 들어, for문 안에 if문이 있으면 indent depth는 2입니다. depth를 줄이는 좋은 방법은 함수(또는 메소드)를 분리하면 됩니다.

  • 하나의 기능당 하니의 pr

  • commit 단위 작게 하기

  • branch 관리 잘하기

2. 이번주 필수 구현사항

  • 깃 정리 (upstream과 맞지 않다..?)
  • 폴더 관리
  • 탭구분 (라우팅 처리)
  • 메뉴 가져오기
  • 메뉴 추가
  • 메뉴 수정
  • 메뉴 삭제
  • git action ci

3. 이번추 추가 구현사항

  • 메뉴를 추가 할때 식당 정보를 입력할수있다.(네이버 or 다음 api 활용)

4. 예상되는 어려움

깃....
api 활용

@hannut91
Copy link
Contributor

크으 스스로에게 제약사항을 많이 거셨네요 ㅎㅎ 그만큼 많이 배우실거에요 ㅎㅎ 그래도 가장 중요한건 기능이 올바르게 되야한다는 걸 잊지 마세요!

@hannut91
Copy link
Contributor

어려움이 있으시면 언제든 DM 주세요

@hannut91
Copy link
Contributor

README를 꾸며주시면 좋을 것 같아요 ㅎㅎ 어떤 앱인지 소개해주세요

@yeonhwakim
Copy link
Collaborator Author

yeonhwakim commented Aug 18, 2020

2주차 후기

배운것

  • 복잡한 커밋을 정리 하는 방법

잘한것

  • 기능별로 pr을 나눴다.
  • commit 단위를 줄였다.

아쉬운점

  • 주 7회 pr을 보내지 못했다.
  • 아직 테스트 코드에 대해 이해를 못하는 부분이있다.
  • 커밋 정리 연습을 못했다
  • git acion ci를 적용하지 못했다.

느낀점

기능별로 pr을 나누고 commit단위를 줄이는데 신경을 썼다.
그러다 보니 자연스럽게 어떻게 개발을 진행해야 될지 정리가 되서 신기했다.
가끔은 흐름이 부자연스러우면 git reset HEAD^를 남발하며 다시 작성하기를 반복했지만....
생각보다 진도를 많이 못나가서 아쉽다.

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

2 participants