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

[FE] Feat(#41): 프로젝트 전체 패키지 구조 세팅 #57

Merged
merged 3 commits into from
Jan 22, 2024

Conversation

saesang
Copy link
Contributor

@saesang saesang commented Jan 21, 2024

#️⃣ 연관된 이슈

#41

Pull Request Type

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 코드 리팩토링
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 테스트 추가, 테스트 리팩토링
  • 코드에 영향을 주지 않는 변경사항
    • 오타 수정, 문서 수정, 변수명 변경, 주석 추가 및 수정

Pull Request Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 변경 사항에 대한 테스트를 모두 통과했나요?
  • 코드 정렬은 적용했나요?
  • 새로운 branch에 작업 후 dev로 PR을 요청했나요?

📝 작업 내용

클린 아키텍처를 적용하여 프로젝트를 모듈화했습니다.

-모듈 설명-

  1. app: 어플 실행 Activity 구현(main)
  2. core: 모든 모듈에서 공통적으로 사용하는 메소드 구현
  3. data: 서버 통신과 관련된 api 및 data model과 로컬 DB 구현(repository, api, data model)
  4. domain: presentation 모듈과 data 모듈을 조작하는 로직 구현(usecase)
  5. presentation: UI 관련 모든 것 구현(View, ViewModel)

💬 리뷰 요구사항

이게 뭐라고,, 너무 힘들었어요,,,,

기존 프로젝트를 지우고 새로 생성했습니다.
Copy link

github-actions bot commented Jan 21, 2024

📝 테스트 커버리지 리포트

Overall Project 85.19% 🍏

There is no coverage information present for the Files changed

Copy link
Contributor

@ghdcksgml1 ghdcksgml1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

대박입니다... 멀티모듈 구조를 하루만에 깨우치시다니...

더 세세한건 프로젝트 진행하면서 수정해도 될 것 같네요 고생하셨습니다 😁😁👍👍👍👍👍

Copy link
Contributor

@jusung-c jusung-c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오.. 멀티 모듈로 진행하시는군요..👍👍
수고하셨습니다!

@rndudals
Copy link
Contributor

고생하셨습니다~!~!

@ghdcksgml1 ghdcksgml1 merged commit bdd1869 into dev Jan 22, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants