Setting/#107 relase, debug 앱 분리 #108
Conversation
Walkthrough이번 PR에서는 여러 설정 파일에 변경사항이 적용되었습니다. IDE 설정 파일에 App Insights의 기본 탭으로 "Firebase Crashlytics"가 지정되었으며, Gradle 빌드 스크립트에 새로운 release 서명 구성과 함께 release 및 debug 빌드 타입의 설정(디버그 여부, manifest 플레이스홀더, applicationIdSuffix, versionNameSuffix 등)이 추가·수정되었습니다. 또한, 디버그 전용 google‑services.json 파일과 AndroidManifest.xml에서 동적 리소스 플레이스홀더를 사용하는 변경, 그리고 디버그용 adaptive 아이콘, 배경 색상, 앱 이름 문자열 리소스가 추가되었습니다. Changes
Assessment against linked issues
✨ Finishing Touches
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
app/build.gradle.kts (1)
18-21: 검토: signingConfigs 블록 구성
릴리즈 서명 구성을 위한signingConfigs블록이 추가되었으나, 현재 해당 블록이 비어 있습니다. 향후 릴리즈 키스토어 설정을 진행할 계획이라면, 관련 주석이나 TODO를 추가해두면 좋겠습니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
app/src/main/ic_toyou_debug-playstore.pngis excluded by!**/*.png
📒 Files selected for processing (8)
.idea/appInsightsSettings.xml(1 hunks)app/build.gradle.kts(2 hunks)app/src/debug/google-services.json(1 hunks)app/src/main/AndroidManifest.xml(1 hunks)app/src/main/res/mipmap-anydpi-v26/ic_toyou_debug.xml(1 hunks)app/src/main/res/mipmap-anydpi-v26/ic_toyou_debug_round.xml(1 hunks)app/src/main/res/values/ic_toyou_debug_background.xml(1 hunks)app/src/main/res/values/strings.xml(1 hunks)
🧰 Additional context used
🪛 Gitleaks (8.21.2)
app/src/debug/google-services.json
18-18: Uncovered a GCP API key, which could lead to unauthorized access to Google Cloud services and data breaches.
(gcp-api-key)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: ci/bitrise/d8a98037-4e20-460c-982d-a2e7e924fabe/pr
🔇 Additional comments (9)
app/src/main/res/values/strings.xml (1)
3-3: 새로운 디버그 앱 이름 문자열 리소스 추가됨
<string name="app_name_debug">투유블랙</string>가 debug 빌드를 위해 적절하게 추가되었습니다.app/src/main/res/values/ic_toyou_debug_background.xml (1)
1-4: 디버그용 배경 색상 리소스 파일 추가됨새로운 XML 파일이 올바르게 구성되었으며,
#2F2F2E값이 디버그 아이콘 배경으로 적절히 설정되었습니다.app/src/main/res/mipmap-anydpi-v26/ic_toyou_debug.xml (1)
1-5: 디버그용 Adaptive Icon 파일 추가됨adaptive icon 구성이 명확하며, 배경과 포그라운드 리소스가 올바르게 참조되고 있습니다.
.idea/appInsightsSettings.xml (1)
3-5: Firebase Crashlytics 기본 탭 설정 옵션 추가됨새로운
<option name="selectedTabId" value="Firebase Crashlytics" />옵션이 추가되어, AppInsightsSettings에서 기본 탭이 Firebase Crashlytics로 설정됩니다.app/src/main/res/mipmap-anydpi-v26/ic_toyou_debug_round.xml (1)
1-5: 검토: 새로운 디버그 라운드 아이콘 XML 파일
이 파일은 새로운 adaptive‑icon XML로, 배경에@color/ic_toyou_debug_background와 전경에@mipmap/ic_toyou_debug_foreground를 사용하고 있습니다. 아이콘 리소스 관리 측면에서 적절하게 구현되었습니다.app/src/main/AndroidManifest.xml (2)
12-14: 검토: 동적 아이콘 및 앱 이름 설정 변경
<application>태그 내에서android:icon,android:label,android:roundIcon속성이 정적 리소스에서 동적 placeholder로 변경되었습니다. 빌드 타입에 따른 유연한 리소스 관리가 가능해졌으므로 적절한 수정입니다.
17-20: 검토: 액티비티 레이블 변경
MainActivity의android:label속성이 제거되었습니다. 어플리케이션 레벨의 레이블 상속 구조를 활용한 것으로 보이며, 디자인 의도에 부합합니다.app/build.gradle.kts (2)
39-50: 검토: Release 빌드 타입 설정
release빌드 타입에서isDebuggable이false로 설정되고, manifestPlaceholders를 통해 정적 리소스 대신 동적 리소스를 사용하도록 구성되었습니다. 다만, 릴리즈 서명 구성으로debug서명 설정을 사용하고 있는 부분은 통합 키스토어 관리 의도로 보이나, 의도한 변경사항인지 한 번 더 검토해 주시기 바랍니다.
51-59: 검토: Debug 빌드 타입 설정
debug빌드 타입에서는isDebuggable, manifestPlaceholders,applicationIdSuffix및versionNameSuffix설정이 명확하게 적용되었습니다. 구성에 문제가 없으며, 변경사항을 승인합니다.
|
Caution No docstrings were generated. |
#️⃣연관된 이슈
📝작업 내용
스크린샷 (선택)
Summary by CodeRabbit
New Features
Chores