Skip to content

maintenance(platform): 쓰지 않는 hilt-navigation-compose 의존과 feature 컨벤션 플러그인을 걷는다 (#2247) - #2257

Merged
1hyok merged 1 commit into
developfrom
1hyok/confident-keller-srwoq8
Oct 5, 2026
Merged

1hyok merged 1 commit into
developfrom
1hyok/confident-keller-srwoq8

Conversation

@1hyok

@1hyok 1hyok commented Oct 5, 2026 •

Copy link
Copy Markdown
Collaborator

읽을 파일: app/build.gradle.kts, feature/setting/presentation/build.gradle.kts, build-logic/build.gradle.kts, build-logic/src/main/kotlin/AndroidFeatureConventionPlugin.kt(삭제)

📌𝘐𝘴𝘴𝘶𝘦𝘴

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

  • app·afternote·receiver·setting 의 implementation(libs.androidx.hilt.navigation.compose) 를 지웠다. 네 모듈은 androidx.hilt.navigation.* 를 main·test·androidTest 어디서도 import 하지 않고, hiltViewModel 은 Hilt 컨벤션이 넣는 hilt-lifecycle-viewmodel-compose 에서 받는다.
  • 적용하는 모듈이 0개인 AndroidFeatureConventionPlugin 을 지우고 build-logic/build.gradle.kts 의 등록과 ExplicitApiModulePolicyKonsistTest 의 afternote.android.feature 항목을 함께 걷었다.
  • ktlint-coverage.mjs 주석의 컨벤션 사슬 예시가 지운 플러그인으로 시작해 남은 사슬(library.compose → library → lint)로 고쳤다.
  • 남긴 것: onboarding 의 같은 줄(담당이 다르다), 버전 카탈로그 항목, afternote.android.navigation 컨벤션.

로컬 검증(origin/develop d5ac051 기준, CI 스텁 설정)

  • assembleDebug :app:compileDebugUnitTestKotlin :app:compileDebugAndroidTestKotlin :app:testDebugUnitTest 와 afternote·receiver·setting presentation 의 testDebugUnitTest, :konsist:test 통과.
  • :build-logic:test 36건 통과. 처음 실행에서 TestKit 빌드가 Maven Central 429 로 2건 실패해 그 2건만 다시 돌려 통과를 확인했다.

🧪 CI Test Plan

{
  "androidTest": {
    "mode": "full",
    "reason": "app/build.gradle.kts 와 build-logic/ 를 바꿔 전역 빌드 변경 규칙상 full 이다. 루트 Nav2 아래 화면의 hiltViewModel 생성이 그대로 동작하는지도 계측 전량으로 확인한다.",
    "tests": []
  }
}

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

화면 변경 없음.

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

…플러그인 제거 (#2247)

app·afternote·receiver·setting 은 androidx.hilt.navigation.* 를 import 하지 않고
hiltViewModel 은 Hilt 컨벤션의 hilt-lifecycle-viewmodel-compose 에서 받는다.
AndroidFeatureConventionPlugin 은 적용하는 모듈이 없어 등록과 konsist 목록 항목까지 걷는다.
@github-actions github-actions Bot added the sibling-conflict 열린 형제 PR 과 같은 파일에서 충돌 — 나중에 머지되는 쪽이 base 를 병합해야 한다. 해소되면 자동으로 떨어진다 label Oct 5, 2026
@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

열린 PR 중 이 PR 과 같은 파일을 서로 다르게 고친 것이 있다. 어느 쪽이 먼저 머지되든 나중 쪽은 base 를 병합해야 통과한다 (#1750).

머지 자체는 GitHub 이 막는다 — 한쪽이 들어가면 다른 쪽은 CONFLICTING 이 되어 merge queue 에 못 들어간다.
이 라벨은 그 전에 순서를 정하라는 신호다. 먼저 갈 쪽을 정하고, 나중 쪽은 그 머지 뒤에 base 를 병합한다.
상대 PR 에도 같은 안내가 달린다.

이 코멘트는 리컨사일러가 매 실행 현재 상태로 고쳐 쓴다. 해소되면 sibling-conflict 라벨은 자동으로 떨어진다.

@github-actions github-actions Bot added the android-test PR head에서 Android Managed Device 계측 테스트 실행 label Oct 5, 2026
@1hyok
1hyok marked this pull request as ready for review October 5, 2026 06:16
@github-actions
github-actions Bot requested a review from Sadturtleman October 5, 2026 06:16
@1hyok
1hyok added this pull request to the merge queue Oct 5, 2026
Merged via the queue into develop with commit e570303 Oct 5, 2026
47 checks passed
@1hyok
1hyok deleted the 1hyok/confident-keller-srwoq8 branch October 5, 2026 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

android-test PR head에서 Android Managed Device 계측 테스트 실행 sibling-conflict 열린 형제 PR 과 같은 파일에서 충돌 — 나중에 머지되는 쪽이 base 를 병합해야 한다. 해소되면 자동으로 떨어진다

Projects

None yet

Development

Successfully merging this pull request may close these issues.

maintenance(platform): 쓰지 않는 hilt-navigation-compose 의존과 미적용 feature 컨벤션 플러그인을 걷는다 (#1702 분리)

2 participants