Skip to content

Comments

[WIP] Assess repository structure and best practices#57

Closed
Copilot wants to merge 1 commit intomainfrom
copilot/evaluate-repository-structure
Closed

[WIP] Assess repository structure and best practices#57
Copilot wants to merge 1 commit intomainfrom
copilot/evaluate-repository-structure

Conversation

Copy link

Copilot AI commented Feb 19, 2026

  • 레포지토리 코드 분석 완료
  • 버그 수정: UserServiceScheduleService에서 EntityExistsExceptionEntityNotFoundException 교체 (엔티티를 찾지 못할 때 잘못된 예외 사용)
  • Swagger 애노테이션 추가: UserControllerScheduleController@Tag / @Operation 누락
  • RegistrationService에서 중복 수강 신청 시 DuplicateResourceException 사용 (현재 IllegalStateException → 의미론적으로 409 Conflict이 맞음)
  • RegistrationService 상수(LOCK_WAIT_TIME, LOCK_LEASE_TIME) 클래스 최상단으로 이동
  • 기존 테스트 통과 확인

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

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

Successfully merging this pull request may close these issues.

2 participants