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

[BE] refactor: review group 관련 예외를 발생하는 위치의 패키지로 이동 #999

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

skylar1220
Copy link
Contributor


🚀 어떤 기능을 구현했나요 ?

  • 몇몇 reviewGroup 관련 예외들이 reviewGroup 패키지 내에서만 발생함에도 불구하고 review 패키지에 위치해있습니다. 따라서 해당 예외의 위치를 reviewGroup 패키지로 이동시켰습니다.

🔥 어떻게 해결했나요 ?

  • reviewGroup 패키지 내에서 발생하는 reviewGroup 관련 예외를 review 패키지에서 reviewGroup 패키지로 이동시켰습니다.

review와 review group 두 패키지에서 모두 참조하는 예외의 경우

  1. reivew group이 review를 참조하는 것과
  2. review가 review group을 참조하는 것 중

2번이 자연스러운 흐름이라고 생각하여 역시 review group 패키지로 이동시켰습니다.

📝 어떤 부분에 집중해서 리뷰해야 할까요?

  • review group이 review에 대한 의존성을 갖지 않는지

📚 참고 자료, 할 말

- review 패키지에서 사용되는 예외도 있으나, review가 reviewGroup을 의존하는 것이 반대의 경우보다 자연스럽기때문에 이동
Copy link

Test Results

157 tests   154 ✅  4s ⏱️
 58 suites    3 💤
 58 files      0 ❌

Results for commit 862fd31.

Copy link
Contributor

@nayonsoso nayonsoso left a comment

Choose a reason for hiding this comment

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

review와 review group 두 패키지에서 모두 참조하는 예외의 경우
reivew group이 review를 참조하는 것과
review가 review group을 참조하는 것 중
2번이 자연스러운 흐름이라고 생각하여 역시 review group 패키지로 이동시켰습니다

동의합니다 😊👍

@skylar1220 skylar1220 merged commit 1306c7b into develop Dec 18, 2024
7 checks passed
@donghoony donghoony deleted the be/refactor/998-move-exception-package branch December 18, 2024 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[BE] 예외를 발생하는 위치의 패키지로 이동한다
4 participants