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

Team 9 Code Review #25

Open
creamchoco3080 opened this issue Nov 21, 2018 · 0 comments
Open

Team 9 Code Review #25

creamchoco3080 opened this issue Nov 21, 2018 · 0 comments

Comments

@creamchoco3080
Copy link

Testing

image

  • Branch Coverage 가 비교적 낮다.
  • 아르바이트 / 고용주 페이지 또한 서비스에 중요한 컴포넌트라 생각되는데 충분히 테스트되지 않았다.
  • unit 테스트가 좋은 convention 을 따른 것 같고 가독성이 있다.

Miscellaneous

  • Repository 에서 travis 를 사용하지만, 세팅만 되어있고 실질적인 테스트를 실행하지 않는다.
  • Wiki 가 현재 버전에 맞게 update 되어있지 않은 것 같다.

Lessons

  • 프로젝트의 complexity 를 줄이기 위해 Timezone 기능을 별도의 서비스로 handling 하였다.
  • 컴포넌트를 세분화해 작성하여 컴포넌트의 재사용성을 높였다.
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

1 participant