Repository navigation
Fix/cd action 파이어베이스 디버그 빌드 생성 액션 환경변수 설정 추가 - #277
Merged
Merged
Conversation
Contributor
Walkthrough안드로이드 CD 워크플로우를 제거하고, 수동 Firebase 배포 워크플로우를 리포맷팅 및 서명 환경 변수를 추가했습니다. 또한 앱 버전을 1.0.1로 업데이트하고 코드 포맷팅을 정리했습니다. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested reviewers
안녕하세요! 이번 PR을 살펴보니 몇 가지 의견을 나누고 싶네요. 먼저 android-cd.yml 워크플로우 제거는 의도적인 선택인 것 같은데, develop 브랜치에 자동 배포가 필요 없어진 건가요? 혹시 모바일 배포 정책이 변경되셨다면 팀원들과 충분히 공유하셨는지 확인해보시면 좋겠습니다. manual-deploy-to-firebase.yml 개선은 정말 좋아 보입니다! 서명 환경 변수( 버전 업데이트 (1.0.0 → 1.0.1)는 마이너 버전 업 정도로 보이는데, 패치 내용이 있으신가요? 그리고 MainActivity.kt의 포맷팅은 코드 정렬 차원에서 좋은 정리인 것 같습니다. 전반적으로 매우 깔끔하고 일관성 있는 변경들이에요. 검토 잘 진행하겠습니다! 😊 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
.github/workflows/manual-deploy-to-firebase.yml (1)
29-34: 서명 비밀값 범위를 빌드 단계로 제한해 주세요.현재
GITHUB_ENV에 넣으면 이후 모든 단계(서드파티 액션 포함)에서 접근 가능해 노출 범위가 넓어집니다. 빌드 단계env로 한정하는 편이 안전합니다.🔐 제안 수정안
- - name: Set signing env vars - run: | - echo "RELEASE_KEYSTORE_PATH=$GITHUB_WORKSPACE/release.keystore" >> $GITHUB_ENV - echo "RELEASE_KEYSTORE_PASSWORD=${{ secrets.RELEASE_KEYSTORE_PASSWORD }}" >> $GITHUB_ENV - echo "RELEASE_KEY_ALIAS=${{ secrets.RELEASE_KEY_ALIAS }}" >> $GITHUB_ENV - echo "RELEASE_KEY_PASSWORD=${{ secrets.RELEASE_KEY_PASSWORD }}" >> $GITHUB_ENV - - name: Build Debug APK only + env: + RELEASE_KEYSTORE_PATH: ${{ github.workspace }}/release.keystore + RELEASE_KEYSTORE_PASSWORD: ${{ secrets.RELEASE_KEYSTORE_PASSWORD }} + RELEASE_KEY_ALIAS: ${{ secrets.RELEASE_KEY_ALIAS }} + RELEASE_KEY_PASSWORD: ${{ secrets.RELEASE_KEY_PASSWORD }} run: ./gradlew assembleDebug🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.github/workflows/manual-deploy-to-firebase.yml around lines 29 - 34, 현재 "Set signing env vars" 스텝이 echo로 값을 GITHUB_ENV에 넣어 전체 후속 단계에 노출시키므로, 서명 비밀값(예: RELEASE_KEYSTORE_PATH, RELEASE_KEYSTORE_PASSWORD, RELEASE_KEY_ALIAS, RELEASE_KEY_PASSWORD)을 GITHUB_ENV 대신 빌드 단계의 env로 제한하도록 변경하세요; 구체적으로 "Set signing env vars" 스텝에서 echo 줄을 제거하고 대신 서명 관련 변수를 실제 빌드 단계(예: 앱 빌드 step)를 호출하는 job/step의 env: 블록에 직접 매핑하여 secrets 사용(${ { secrets.RELEASE_KEYSTORE_PASSWORD }} 등)으로 전달되게 하며, 필요한 경우 RELEASE_KEYSTORE_PATH는 런타임에서 $GITHUB_WORKSPACE/release.keystore로 계산하여 빌드 step의 env에 설정하세요.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.github/workflows/manual-deploy-to-firebase.yml:
- Line 27: Replace uses of echo when feeding base64 content into base64 -d with
printf '%s' to avoid added newlines and ensure safe decoding; specifically
update the command that writes to debug.keystore (the invocation referencing
secrets.DEBUG_KEYSTORE_BASE64) and the second occurrence noted in the comment
(the other base64 secret on line 39) to use printf '%s' and ensure the secret
variable is quoted when passed to printf. Ensure both invocations replace echo
with printf '%s' and wrap the secret expansion in quotes so decoding is stable
and safe.
---
Nitpick comments:
In @.github/workflows/manual-deploy-to-firebase.yml:
- Around line 29-34: 현재 "Set signing env vars" 스텝이 echo로 값을 GITHUB_ENV에 넣어 전체 후속
단계에 노출시키므로, 서명 비밀값(예: RELEASE_KEYSTORE_PATH, RELEASE_KEYSTORE_PASSWORD,
RELEASE_KEY_ALIAS, RELEASE_KEY_PASSWORD)을 GITHUB_ENV 대신 빌드 단계의 env로 제한하도록 변경하세요;
구체적으로 "Set signing env vars" 스텝에서 echo 줄을 제거하고 대신 서명 관련 변수를 실제 빌드 단계(예: 앱 빌드
step)를 호출하는 job/step의 env: 블록에 직접 매핑하여 secrets 사용(${ {
secrets.RELEASE_KEYSTORE_PASSWORD }} 등)으로 전달되게 하며, 필요한 경우 RELEASE_KEYSTORE_PATH는
런타임에서 $GITHUB_WORKSPACE/release.keystore로 계산하여 빌드 step의 env에 설정하세요.
ℹ️ Review info
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
.github/workflows/android-cd.yml.github/workflows/manual-deploy-to-firebase.ymlapp/build.gradle.ktsapp/src/main/java/com/emotionstorage/emotionstorage/MainActivity.kt
💤 Files with no reviewable changes (1)
- .github/workflows/android-cd.yml
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
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 by CodeRabbit