-
Notifications
You must be signed in to change notification settings - Fork 0
[Refactor/#175] 빌드 성능 개선 작업을 진행합니다. #176
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
base: develop
Are you sure you want to change the base?
Conversation
- Gradle 빌드 옵션(병렬 처리, 캐싱, JVM args)을 수정하여 빌드 성능을 개선
개요빌드 워크플로우를 간소화하고 Gradle 성능 설정을 최적화하며 Kotlin 빌드 산물을 제외하도록 구성을 업데이트합니다. 변경 사항
예상 코드 리뷰 노력🎯 2 (Simple) | ⏱️ ~12분 축하 시
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. 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 |
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.
Actionable comments posted: 2
🤖 Fix all issues with AI agents
In @.github/workflows/develop_branch.yml:
- Around line 29-30: Quote the environment variable and avoid the intermediate
file: when writing the keystore use the quoted variable "$RELEASE_STORE_BASE_64"
and pipe it directly into base64 --decode to write release.jks, replacing the
two-step echo > encoded_keystore.txt and base64 --decode encoded_keystore.txt
flow; reference the existing echo, encoded_keystore.txt, base64 --decode and
release.jks symbols when making the change.
In `@gradle.properties`:
- Line 5: 현재 gradle.properties의 org.gradle.configuration-cache=true 설정은 사용 중인
플러그인(AGP 8.10.1, Kotlin 2.1.20, KSP 2.1.20-1.0.32)과 호환되지만 일부 KSP 엣지케이스가 있을 수
있으니, 로컬과 CI에서 전체 빌드(gradle clean build) 및 구성 캐시 활성화 모드(./gradlew build
--configuration-cache)를 실행해 문제 여부를 확인하고, 실패 시 CI의 Gradle 호출에
--no-configuration-cache 플래그를 추가하거나 gradle.properties의
org.gradle.configuration-cache 값을 false로 되돌리는 방법으로 일시 해제하며 결과를 PR 설명에 기록하고 재현
방법을 남겨 주세요.
🧹 Nitpick comments (1)
.gitignore (1)
48-49:.kotlin만으로 디렉토리와 파일 모두 매칭 가능합니다.
.kotlin패턴은 gitignore에서 파일과 디렉토리 모두를 매칭합니다. 따라서.kotlin/(Line 48)는.kotlin(Line 49)에 의해 이미 포함되므로 중복입니다. 큰 문제는 아니지만 참고 부탁드립니다.
[ PR Content ]
빌드 성능 개선 작업을 진행했습니다.
작업 관련 내용은 노션에 따로 정리했습니다~! -> 노션링크
Related issue
Screenshot 📸
Work Description
To Reviewers 📢
Summary by CodeRabbit
릴리스 노트