-
Notifications
You must be signed in to change notification settings - Fork 2
fix: 주석 추가 #98
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
Merged
Merged
fix: 주석 추가 #98
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Docker 빌드 시 테스트 및 ktlint 스킵 설정 - 테스트 환경에서 Redis Bean 문제 해결 - TestConfig 추가하여 Mock RedisConnectionFactory 제공 - application-test.yml에 Redis AutoConfiguration 제외 설정 - 모든 테스트 클래스에 @import(TestConfig::class) 적용 - Ktlint 스타일 수정 - RedisConfig, SecurityConfig, AiChatController 포맷팅 수정 - AppConfig 파일 끝 개행 추가 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Spring AI was attempting to execute H2-specific schema (schema-h2.sql) on PostgreSQL, causing infinite restart loop due to unsupported CLOB type. Changes: - Set spring.sql.init.mode to never in application.yml - Add spring.ai.vectorstore.jdbc.initialize-schema: false in application-prod.yml - Remove schema-locations pointing to H2 schema Fixes database initialization error: type "clob" does not exist 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Added CONTAINER_2_NAME environment variable to support proper Blue/Green deployment. Without this, the GREEN variable was empty, causing containers to be created with random names instead of team11_1/team11_2. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Added /actuator/health to permitAll in SecurityConfig to enable container health checks during Blue/Green deployment. Without this, health checks fail due to authentication requirement, causing deployment rollback. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Temporarily permit /weather/test1 endpoint to test Redis caching without authentication in production environment. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
# Conflicts: # src/test/kotlin/com/back/koreaTravelGuide/domain/ai/tour/client/TourApiClientTest.kt
- UserRepository.findByRoleAndLocationContains -> findByRoleAndLocation 변경 - location 파라미터 타입: String -> Region enum으로 변경 - GuideService에서 String을 Region enum으로 변환하는 로직 추가 - upstream merge 후 발생한 타입 불일치 해결을 위한 임시 수정 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
YangHJ2415
approved these changes
Oct 10, 2025
Collaborator
YangHJ2415
left a comment
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.
확인 했습니다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Region.values()→Region.entries로 업데이트 (Kotlin 1.9+ 권장)Changes
findByRoleAndLocationContains→findByRoleAndLocation으로 변경Region.values()→Region.entries업데이트Test plan
🤖 Generated with Claude Code