#1 - 프로젝트 초기 세팅 #2
Merged
Conversation
This file contains 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
sksowk156
reviewed
Dec 8, 2024
| } | ||
| } | ||
|
|
||
| internal fun Project.configureHiltAndroid() { |
There was a problem hiding this comment.
혹시 이 확장 함수를 다른 파일에서도 쓰는건가요?
There was a problem hiding this comment.
현재로썬 그렇지 않아용...!
Droidknights 레포지토리 참고해서 만들다보니 요롷게 되었군요
저 부분 굳이 함수로 안빼구 그냥 그레이들에 넣어도 되긴 해요
| internal fun Project.configureKotlinAndroid() { | ||
| // with(plugins) { | ||
| // apply("com.google.gms.google-services") | ||
| // } |
There was a problem hiding this comment.
넵 나중에 파이어베이스 연동할 때 주석부분 다 풀면 되어용
| // implementation(project(":core:analytics")) | ||
| // implementation(project(":core:common-ui")) | ||
|
|
||
| val libs = project.extensions.libs |
There was a problem hiding this comment.
libs 가 여러 군데서 쓰이는걸 보면 이것도 글로벌하게 뺄 수 있을 것 같은데 혹시 어떻게 생각하시나요?
There was a problem hiding this comment.
글로벌로 빼면 사용하지 않아도 메모리를 계속 잡고있게 되어요.
정확히 gradle이 어떤 생명주기를 가지는 진 모르겠지만 싱글톤으로 있는 것 보다는 gc에 의해서 메모리가 수거될 수 있도록 최대한 스코프를 좁히는 것이 좋을 것 같아요..!
There was a problem hiding this comment.
그렇네요 저는 원래 3번이상 중복되면 스코프를 한단계씩 빼두는 습관이 있는데 extensions.libs 자체가 무거운 작업이 아니니 굳이 글로벌하게 뺄 필요는 없을 것 같긴하네요 👍
|
|
||
| ## AndroidX | ||
| # https://developer.android.com/jetpack/androidx/releases/core | ||
| androidxCore = "1.13.1" |
There was a problem hiding this comment.
이야 링크까지.... 키야 진짜 섬세하십니다 👍 👍
tgyuuAn
added
머지 해도될듯염🌟
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.
1. ⭐️ 변경된 내용
2. 🖼️ 스크린샷(선택)
3. 💡 알게된 부분
4. 📌 이 부분은 꼭 봐주세요!
일단 앱 이름이 정해져있지 않아 리스트업에서 "채움" 이라는 이름이 마음에 들어서 그렇게 설정했습니다.
추후에 이름 정해지면 그걸로 싹 바꿀게용
작업하시기 전에
./gradlew addKtlintCheckGitPreCommitHook를 터미널에 꼭 입력하고 작업해주세욥커밋할 때 마다 ktlintCheck하는 커맨드 입니다!