[REFACTOR/#302] 빌드로직 구현 및 구조 수정 #303
Conversation
There was a problem hiding this comment.
이런 확장함수들은 따로 패키지화 하는건 어떨까요??
There was a problem hiding this comment.
좋은 생각인 것 같아요! 해당 파일도 파일 이름을 명확하게 해주면 좋을 것 같아서 AndroidExtension으로 바꿔서 extension 패키지 안에 넣어두었어요!
There was a problem hiding this comment.
그리고 extension이라는 패키지를 만들면서 이 패키지에 들어가지 않는 파일들은 convention패키지를 만들어 안에 넣어두었어요! 확인부탁드립니당
There was a problem hiding this comment.
이 파일도 확장함수 패키지 안에 같이 넣어주는거 어떤가요?
| implementation(project(":core:analytics")) | ||
|
|
||
| // navigation | ||
| implementation(libs.findLibrary("hilt.navigation.compose").get()) |
There was a problem hiding this comment.
ComposeAndroid.kt 플러그인을 terning.android.compose id를 사용하여 적용하는거 같은데, 해당 라이브러리가 이미 해당 파일 안에 있어요! 한 곳에서는 제거하는게 좋을 것 같습니다!!
There was a problem hiding this comment.
오호.. 안 그래도 해당 라이브러리를 넣지 않아도 문제가 없었는데 그런 이유가 있었군요! 알려주셔서 감사해요:)
| implementation(platform(libs.okhttp.bom)) | ||
| implementation(libs.okhttp) | ||
| implementation(libs.okhttp.logging) | ||
| implementation(libs.retrofit.core) | ||
| implementation(libs.retrofit.kotlin.serialization) | ||
| implementation(libs.kotlinx.serialization.json) | ||
| implementation(libs.retrofit2.kotlinx.serialization.converter) |
There was a problem hiding this comment.
레트로핏 관련해서도 플러그인을 만들어두면 여기랑 terning.android.data.gradle.kts에서 활용이 가능할 거 같아요!
There was a problem hiding this comment.
오 좋은 아이디어인데요? 이거는 제가 빌드로직 조금 더 공부해보고 적용해보도록 할게요..!!
| defaultConfig { | ||
| buildConfigField("String", "VERSION_NAME", "\"${libs.versions.versionName.get()}\"") | ||
| } | ||
| buildFeatures { | ||
| buildConfig = true | ||
| } |
There was a problem hiding this comment.
다른 feature gradle에는 없던데 여기엔 왜 있는건가요??
There was a problem hiding this comment.
feature/mypage에서 BuildConfig클래스를 사용하기 때문입니다!
마이페이지에서 버전 정보를 표시해줘야 하기 때문에 넣어주었어요!
⛳️ Work Description
추가로 해야 할 일
📸 Screenshot
📢 To Reviewers