-
Notifications
You must be signed in to change notification settings - Fork 1
[#332] Fastlane 코드 사이닝 세팅 완료 #334
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
Summary of ChangesHello @Hrepay, 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은 iOS 프로젝트의 코드 사이닝 및 빌드 구성을 Fastlane을 활용하여 자동화하고 표준화하는 것을 목표로 합니다. 개발 및 배포 환경에 따라 Entitlements 파일을 분리하고, Project.swift에 명시적인 빌드 설정을 추가하며, Fastlane 워크플로우를 재정비하여 온보딩 및 배포 프로세스의 효율성을 높였습니다. Highlights
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
|
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은 Fastlane을 사용한 코드 사이닝 설정을 자동화하는 중요한 변경사항을 담고 있습니다. 개발 및 배포 환경에 따라 Entitlements 파일을 분리하고, Fastlane 설정을 업데이트하여 배포 과정을 개선한 점이 좋습니다.
다만, 몇 가지 개선점을 제안합니다. Project.swift 파일에서 빌드 설정이 여러 타겟에 걸쳐 중복되고 있으며, 이 과정에서 기존의 중요한 빌드 설정이 누락되는 critical한 문제가 발견되었습니다. 이를 해결하기 위해 공통 설정을 추출하여 재사용하는 리팩토링을 제안했습니다. 또한, Appfile과 Matchfile에 하드코딩된 민감한 정보(Apple ID, Team ID, Git 저장소 URL)를 보안 강화를 위해 환경 변수로 관리하도록 수정하는 것을 권장합니다.
자세한 내용은 각 파일에 남긴 코멘트를 참고해주세요.
Funital
left a comment
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.
그럼 이제 프로비저닝 파일을 주고받는 번거로운 작업이 없어진건가요!?
넹, 저번과 동일하게 devleop 하고 appstore용 match 파일만 땡겨오시면 됩니다. |
#️⃣ 관련 이슈
Resolved #332
💡작업 내용
Entitlements 및 빌드 구성 분리
빌드 설정 및 코드 사이닝
Fastlane 자동화 및 문서 개선
💬리뷰 요구사항(선택)