Conversation
Walkthrough새로운 추상 기본 클래스 BaseEntity를 도메인 모델에 추가하고, Spring Boot 설정 파일(application.yml)을 메인 및 테스트 환경용으로 작성했습니다. 기존 properties에서 애플리케이션 이름 설정을 제거하고 YAML 형식으로 통합했습니다. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@src/main/resources/application.yml`:
- Around line 11-15: Change the default JPA hibernate ddl setting from update to
a safe default (e.g., jpa.hibernate.ddl-auto: validate or none) and remove any
hard-coded update in the main application.yml; instead allow environment
override via SPRING_JPA_HIBERNATE_DDL_AUTO so development can opt into update,
ensuring production uses validate/none by default. Locate the
jpa.hibernate.ddl-auto property in the configuration and replace its value
accordingly, and document/ensure environments set
SPRING_JPA_HIBERNATE_DDL_AUTO=update only for dev.
🧹 Nitpick comments (1)
src/main/java/flipnote/group/domain/model/BaseEntity.java (1)
1-4: 엔티티를 확장할 때@MappedSuperclass적용 필요현재 비어있는 추상 클래스로 사용할 경우 문제없습니다. 다만 이 클래스를 상속받는 엔티티가 생기고 공통 ID/감사 필드 등을 추가할 때는
@MappedSuperclass어노테이션을 적용해야 JPA가 올바르게 처리합니다. Spring Boot 4.0.1은 Jakarta EE를 사용하므로jakarta.persistence.MappedSuperclass를 임포트하면 됩니다.
Summary by CodeRabbit
릴리스 노트
설정
개선