fix: 1.4.3 공휴일 표시 핫픽스#315
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이번 핫픽스는 캘린더 공휴일 표시 기능을 복구하기 위한 업데이트입니다. 공휴일 API 키 주입 누락 문제를 해결하기 위해 빌드 스크립트의 유효성 검사를 강화하고, 관련 프로젝트 설정 및 앱 버전을 최신화하였습니다. Highlights
Ignored Files
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize the Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counterproductive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the project to version 1.4.3, including a fix for holiday display and the integration of a new service key (DATA_GO_KR_SERVICE_KEY) across environment configurations. Feedback focuses on preventing xcconfig parsing errors by escaping potential slashes in the service key, maintaining naming consistency with environment-specific suffixes, and ensuring the example target's Release configuration uses the appropriate production xcconfig instead of the development one.
| check_env_var "GOOGLE_CLIENT_ID_DEV" || return 1 | ||
| check_env_var "GOOGLE_REVERSED_CLIENT_ID_DEV" || return 1 | ||
| check_env_var "KAKAO_NATIVE_APP_KEY_DEV" || return 1 | ||
| check_env_var "DATA_GO_KR_SERVICE_KEY" || return 1 |
There was a problem hiding this comment.
xcconfig 파일에서 //는 주석의 시작으로 인식됩니다. 공공데이터포털의 서비스 키(Service Key)는 Base64 형식이 포함될 수 있어 // 문자가 나타날 가능성이 높습니다. 만약 키에 //가 포함되면 xcconfig 파싱 시 그 이후의 값은 주석으로 처리되어 키가 잘리게 되고, 결과적으로 런타임에 공휴일 API 호출이 실패하게 됩니다. 이 변수를 사용하는 cat 블록 이전에 //를 /$(EMPTY)/와 같이 이스케이프 처리하는 로직을 추가하는 것을 권장합니다. (이 이슈는 Dev, Stage, Prod 모든 환경 설정에 해당합니다.)
| check_env_var "GOOGLE_CLIENT_ID_PROD" || return 1 | ||
| check_env_var "GOOGLE_REVERSED_CLIENT_ID_PROD" || return 1 | ||
| check_env_var "KAKAO_NATIVE_APP_KEY_PROD" || return 1 | ||
| check_env_var "DATA_GO_KR_SERVICE_KEY" || return 1 |
There was a problem hiding this comment.
현재 DATA_GO_KR_SERVICE_KEY는 모든 환경(Dev, Stage, Prod)에서 동일한 환경 변수 이름을 사용하고 있습니다. 이 스크립트의 다른 변수들(GOOGLE_CLIENT_ID_PROD, KAKAO_NATIVE_APP_KEY_PROD 등)은 환경별 접미사를 사용하여 개별적으로 관리되고 있습니다. 일관성을 유지하고 환경별로 다른 키를 사용할 수 있도록 DATA_GO_KR_SERVICE_KEY_PROD와 같은 네이밍 규칙을 따르는 것이 유지보수 측면에서 좋습니다.
| settings: .standard( | ||
| configurations: [ | ||
| .debug(name: "Debug", xcconfig: .relativeToRoot("Config/Dev.xcconfig")), | ||
| .release(name: "Release", xcconfig: .relativeToRoot("Config/Dev.xcconfig")) |
There was a problem hiding this comment.
CalendarFeatureExample 타겟의 Release 구성(Configuration)에서 Dev.xcconfig를 사용하도록 설정되어 있습니다. 일반적으로 Release 빌드는 프로덕션 환경에 가까운 설정을 테스트하기 위해 사용되므로, Prod.xcconfig 또는 Stage.xcconfig를 사용하도록 설정하는 것이 더 적절합니다.
|
✅ PR 체크 성공 모드: 🔨 전체 빌드 📊 모듈별 커버리지
상세 정보
|
1 participant
요약
검증
배포