-
Notifications
You must be signed in to change notification settings - Fork 0
[Feat] 디버그와 릴리즈 앱 동시 설치 가능 #328
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
Conversation
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.
Summary of Changes
Hello @PeraSite, 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은 개발 및 테스트 효율성을 높이기 위해 디버그 버전과 릴리즈 버전의 애플리케이션을 단일 기기에 동시에 설치할 수 있도록 빌드 구성을 개선합니다. 이를 통해 개발자는 두 환경을 쉽게 전환하며 테스트할 수 있습니다.
Highlights
- 디버그 및 릴리즈 앱 동시 설치 가능: 디버그 빌드와 릴리즈 빌드를 같은 기기에 동시에 설치할 수 있도록
applicationIdSuffix를 설정했습니다. - 디버그 앱 이름 설정: 디버그 빌드 전용 앱 이름을 "EAT-SSUDEV"로 설정하여 릴리즈 버전과 구분했습니다.
- 빌드 설정 변경:
app/build.gradle.kts파일에서 디버그 빌드의applicationIdSuffix주석을 해제했습니다. - 기능 확인: 카카오톡 로그인 및 데이터 불러오기 기능이 디버그 및 릴리즈 모드에서 모두 정상 작동함을 확인했습니다.
Using Gemini Code Assist
The 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 in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.
| Feature | Command | Description |
|---|---|---|
| Code Review | /gemini review |
Performs a code review for the current pull request in its current state. |
| Pull Request Summary | /gemini summary |
Provides a summary of the current pull request in its current state. |
| Comment | @gemini-code-assist | Responds in comments when explicitly tagged, both in issue comments and review comments. |
| Help | /gemini help |
Displays a list of available commands. |
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 .gemini/ folder in the base of the repository. Detailed instructions can be found here.
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
-
Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩
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.
Code Review
이 PR은 디버그 빌드와 릴리즈 빌드를 동시에 설치할 수 있도록 applicationIdSuffix를 설정하고, 디버그용 앱 이름을 별도로 지정하는 변경 사항을 담고 있습니다. 이를 통해 개발 편의성이 향상될 것으로 기대됩니다. 변경된 내용은 명확하며 올바르게 구현되었습니다.
다만, 코드 변경 범위 밖의 내용이지만 한 가지 중요한 점을 공유드립니다. app/build.gradle.kts 파일의 release 빌드 설정에서 isMinifyEnabled와 isShrinkResources가 false로 설정되어 있습니다. 프로덕션 앱의 보안을 강화하고 앱 크기를 최적화하기 위해 이 옵션들을 true로 설정하는 것을 강력히 권장합니다. 코드 난독화(obfuscation)와 리소스 축소는 릴리즈 빌드에 필수적인 과정입니다. 이 부분을 별도의 PR에서 다루어 주시면 좋겠습니다.
저도 이 부분에 대해서 조금 의아했는데, 명시적으로 false라고 #311 에서 수정된 것으로 보아 이유가 있는 것 같아보여 냅뒀어요. |
|
옛날 파일을 사용해서 발생하던 빌드 오류 해결하려고 Github Actions의 Secret에서 GOOGLE_SERVICE 값도 수정해두었습니다! |
난독화 작업이 이루어지지 않은 상태에서 저 값만 true로 바꾸었다가... 앱이 정상 작동되지 않는 오류가 발생하여서 급히 바꾸었습니다. 앱 사이즈가 많이 크긴 해서 난독화 작업을 하면 좋을 것 같습니다! |
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.
없슈 좋은데요???
그리고 디버그용 앱 아이콘은 검은 바탕에 로고로 해주시면 좋을 것 같습니다! 앱 아이콘이 같으면 잘 분간하기 어려울 것 같아서욥! 피그마에 없다면 나령님께 요청하면 될 것 같아요. (제가 하려고 했는데, 생각해보니 개발자들 Edit 권한을 빼서... )
그리구 PR 본문에 앱 두개 설치된 스크린샷 첨부해주시면 좋을 것 같슴돠
- assignee는 작업한 사람이에요! 본인을 넣어주면 됩니당
확인했습니다! 이미지는 피그마에서 Export는 되길래 해서 수정한 후 올려놨습니다. |
|
추가적으로 안드로이드 아이콘이 너무 작게 보이길래 확인해보니까 적응형 아이콘 이라는 것을 적용해야 하더라구요. 지금처럼 이미지 크기 하나로만 만들면 아이콘에 패딩이 보입니다... 이건 따로 PR 올려볼게요! |
헐!!! 이전에는 흰색이라서 티가 안나서 몰랐어요... 좋습니다 |
Summary
디버그 빌드와 릴리즈 빌드를 동시에 설치할 수 있게 수정했습니다!
Describe your changes
빌드 관련 설정 변경
Issue
To reviewers
FIrebase에서 debug용 App을 하나 추가했습니다. 정상적으로 사용하시려면 하단 링크에서 google-services.json을 받아 기존 파일과 교체하시면 됩니다.
기존 파일이 1KB, 새 파일이 디버그 관련 데이터가 추가되어 1.8KB인게 맞습니다.
https://www.notion.so/eat-ssu/1d2eeef75a16819a8bc1c84f07e1deb2?source=copy_link#261eeef75a16806cb5c1cf38ac2f56af
퍼블릭 레포지토리라 문제가 생길 수도 있어, 노션 링크로 첨부하는 점 양해 부탁드려요!
디버그용 앱 이름을 없슈, EAT-BUG 등 고심했는데 깔끔한 EAT-SSUDEV로 했습니당...
디버그/릴리즈 두 모드에서 카카오톡 로그인, 데이터 불러오기 등 정상 작동하는 것 확인했습니다!