[Feat] 서명된 release apk를 추출하도록 CI 코드 수정 #451
Conversation
Summary of ChangesHello @kangyuri1114, 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! 이 PR은 CI 환경에서 빌드된 release APK가 서명되지 않아 Firebase App Distribution을 통한 설치에 실패하는 문제를 해결합니다. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. 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 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 counter productive. 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. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. 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
The pull request introduces a new ciRelease signing configuration in app/build.gradle.kts that dynamically retrieves keystore details from environment variables (KEYSTORE_FILE, KEYSTORE_PASSWORD, KEY_ALIAS, KEY_PASSWORD). The release build type is then updated to conditionally use this ciRelease signing configuration when the KEYSTORE_FILE environment variable is set, facilitating signing in CI environments. There are no review comments provided for these changes.
|
✅ CI 완료
|
|
✅ CI 완료
|
|
✅ CI 완료
|
|
✅ CI 완료
|
|
✅ CI 완료
|
HI-JIN2
left a comment
HI-JIN2
left a comment
There was a problem hiding this comment.
App Tester 동작 확인했습니다! 아주 잘됩니다!! 고생 많으셨어요.
그런데 제훈님한테 .debug apk 설치를 분리해달라고 했던거는, 한 휴대폰에서 릴리즈 모드 앱과 디버그 앱을 동시에 설치하기 위함이었어요. 그런데 지금은 서명된 release만 만드는걸로 보여서 이 부분은 함께 얘기해봐야할 것 같습니다! @PeraSite
제 생각에는 Firebase App Distribution 통해서 만드는 CICD는 배포전 debug용 apk를 만들어서 팀내 자체 QA를 하는 목적이 크다고 생각합니다. 이 과정에서 테스트용 DB입력이 이루어지고요.
- 한 휴대폰에서 릴리즈/디버그 둘다 존재해야함
- 디버그용 DB 입력이 있음
두가지를 고려했을때 하나의 Firebase App = 하나의 패키지명(applicationId)을 기준으로 동작해서 하나만 만들 수 있다면 둘중 디버그 모드를 하는게 차라리 맞는 것 같고
릴리즈 모드도 필요하다면 Firebase App을 각각 등록하는게 맞다는 생각이 듭니다.
그리고 혹시 해당 크래시가 CICD 관련 문제였을까요?
|
@HI-JIN2 |
Summary
서명된 release apk를 추출하도록 CI 코드 수정
Describe your changes
기존 anroid.yml CI 코드로 firebase app distribution를 통해 메일로 release, debug apk 배포 시 설치가 안되는 문제가 있었습니다.
Debug가 실패한 이유 : 기존에 설치된 debug app이 있어서 삭제 후 설치하니 됨
Release가 실패한 이유 : 서명되지 않은 apk 파일이었어서
이 이유로 확인되어 release apk 발행 시 github secrets에 있는 keystore 파일 정보를 활용해 서명된 apk 파일이 추출될 수 있게 수정
Issue
To reviewers
바뀐 부분
바뀐 점 : debug apk 파일 추출 삭제, release apk 정상 메일 전송 및 정상 실행 확인