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

[REFACTOR] CertificationController & GithubController Facade &DCI 패턴 적용 #239

Merged
merged 10 commits into from
Aug 12, 2024

Conversation

SSung023
Copy link
Contributor

PR 타입(하나 이상의 PR 타입을 선택해주세요)

☑ 기능 추가
□ 기능 삭제
☑ 리팩토링
□ 버그 수정
□ 의존성, 환경 변수, 빌드 관련 코드 업데이트


반영 브랜치

refactor/236-Certification-Facademain


변경 사항

  • Certification에 Facade 패턴 적용: 인터페이스/구현체 적용
  • CertificationFacade 테스트 코드에 DCI 패턴 적용
  • Github에 Facade 패턴 적용: 인터페이스/구현체 적용
  • GithubFacade 테스트 코드에 DCI 패턴 적용

테스트 결과

image


연관된 이슈

#236


리뷰 요구사항(선택)

SSung023 added 10 commits August 6, 2024 14:00
- 파사드 패턴 적용에 따라 인터페이스 추가 및 구현체 적용
- 이름 변경
- 파사드 패턴 적용에 따른 인터페이스 및 구현체 적용
- 이름 변경
- InstanceProvider 의존에서 InstanceService 의존으로 변경
- Facade에서 Service만 의존하도록 리팩토링
- 깃허브와 관련된 부분은 GithubService로 이동
- 도메인에 해당하는 비지니스 로직 이동
@SSung023 SSung023 added 💡feature 기능 구현, 개선 사항 관련 ⚙️ refactor 리팩토링 🧑🏻‍💻 BE 백엔드 관련 코드 🎮 test 테스트 코드 관련 labels Aug 12, 2024
@SSung023 SSung023 requested a review from kimdozzi August 12, 2024 02:43
@SSung023 SSung023 self-assigned this Aug 12, 2024
@SSung023 SSung023 linked an issue Aug 12, 2024 that may be closed by this pull request
1 task
Copy link

Test Results

158 files  158 suites   45s ⏱️
295 tests 295 ✅ 0 💤 0 ❌
303 runs  303 ✅ 0 💤 0 ❌

Results for commit e8366ac.

Copy link
Contributor

@kimdozzi kimdozzi left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!!

@SSung023 SSung023 merged commit 909ce59 into main Aug 12, 2024
2 checks passed
@SSung023 SSung023 deleted the refactor/236-Certification-Facade branch August 12, 2024 05:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧑🏻‍💻 BE 백엔드 관련 코드 💡feature 기능 구현, 개선 사항 관련 ⚙️ refactor 리팩토링 🎮 test 테스트 코드 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[REFACTOR] CertificationController & GithubController Facade &DCI 패턴 적용
2 participants