Repository navigation
maintenance(platform): 쓰지 않는 hilt-navigation-compose 의존과 feature 컨벤션 플러그인을 걷는다 (#2247) - #2257
Merged
Merged
Conversation
…플러그인 제거 (#2247) app·afternote·receiver·setting 은 androidx.hilt.navigation.* 를 import 하지 않고 hiltViewModel 은 Hilt 컨벤션의 hilt-lifecycle-viewmodel-compose 에서 받는다. AndroidFeatureConventionPlugin 은 적용하는 모듈이 없어 등록과 konsist 목록 항목까지 걷는다.
Contributor
|
열린 PR 중 이 PR 과 같은 파일을 서로 다르게 고친 것이 있다. 어느 쪽이 먼저 머지되든 나중 쪽은 base 를 병합해야 통과한다 (#1750).
머지 자체는 GitHub 이 막는다 — 한쪽이 들어가면 다른 쪽은 CONFLICTING 이 되어 merge queue 에 못 들어간다. 이 코멘트는 리컨사일러가 매 실행 현재 상태로 고쳐 쓴다. 해소되면 |
1hyok
marked this pull request as ready for review
October 5, 2026 06:16
Sadturtleman
approved these changes
Oct 5, 2026
This file contains hidden or 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
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.
읽을 파일:
app/build.gradle.kts,feature/setting/presentation/build.gradle.kts,build-logic/build.gradle.kts,build-logic/src/main/kotlin/AndroidFeatureConventionPlugin.kt(삭제)📌𝘐𝘴𝘴𝘶𝘦𝘴
📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯
implementation(libs.androidx.hilt.navigation.compose)를 지웠다. 네 모듈은androidx.hilt.navigation.*를 main·test·androidTest 어디서도 import 하지 않고,hiltViewModel은 Hilt 컨벤션이 넣는hilt-lifecycle-viewmodel-compose에서 받는다.AndroidFeatureConventionPlugin을 지우고build-logic/build.gradle.kts의 등록과ExplicitApiModulePolicyKonsistTest의afternote.android.feature항목을 함께 걷었다.ktlint-coverage.mjs주석의 컨벤션 사슬 예시가 지운 플러그인으로 시작해 남은 사슬(library.compose → library → lint)로 고쳤다.afternote.android.navigation컨벤션.로컬 검증(origin/develop d5ac051 기준, CI 스텁 설정)
assembleDebug:app:compileDebugUnitTestKotlin:app:compileDebugAndroidTestKotlin:app:testDebugUnitTest와 afternote·receiver·setting presentation 의testDebugUnitTest,:konsist:test통과.:build-logic:test36건 통과. 처음 실행에서 TestKit 빌드가 Maven Central 429 로 2건 실패해 그 2건만 다시 돌려 통과를 확인했다.🧪 CI Test Plan
{ "androidTest": { "mode": "full", "reason": "app/build.gradle.kts 와 build-logic/ 를 바꿔 전역 빌드 변경 규칙상 full 이다. 루트 Nav2 아래 화면의 hiltViewModel 생성이 그대로 동작하는지도 계측 전량으로 확인한다.", "tests": [] } }📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵
화면 변경 없음.
💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴