-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat#109/ 멀티 모듈 구성 #110
feat#109/ 멀티 모듈 구성 #110
Conversation
Risk Level 3 - /home/runner/work/Team2-Dari/Team2-Dari/backend/api-server/src/main/java/com/wootecam/festivals/domain/purchase/service/PurchaseService.java Ensure that the purchase logic handles edge cases, such as concurrent purchases. Risk Level 3 - /home/runner/work/Team2-Dari/Team2-Dari/backend/api-server/src/main/java/com/wootecam/festivals/domain/checkin/service/CheckinService.java The comment regarding the check-in process should be updated to reflect the current implementation. Risk Level 5 - /home/runner/work/Team2-Dari/Team2-Dari/backend/api-server/src/main/java/com/wootecam/festivals/global/config/CloudConfiguration.java API keys and secrets are present in plain text. Consider using environment variables or a secure vault. 🔍 Powered by Code Review GPT |
jacoco, rest-docs 등 공통적인 것은 root 에서 관리하고 모듈별로 의존성 따로 관리하도록 분리
Main.class -> QueueApplication.class
jacoco 리포트 생성 위해 추가
두개의 Job 으로 각각 요청하면 토큰 유효성 에러가 나기 때문에 한번에 올리도록 수정
Code Coverage Report
Files
|
This reverts commit fc9a46d.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다 !
📄 작업 설명
멀티모듈 구성 완료하였습니다.
🚨 관련 이슈
closes #109
🌈 작업 상황
📌 기타