Skip to content

build: build-logic 컨벤션 플러그인 도입 및 전 모듈 적용#6

Merged
seunghee17 merged 2 commits into
developfrom
chore/build-logic
Jul 23, 2026
Merged

build: build-logic 컨벤션 플러그인 도입 및 전 모듈 적용#6
seunghee17 merged 2 commits into
developfrom
chore/build-logic

Conversation

@soyeonLee126

Copy link
Copy Markdown
Collaborator

작업 개요

멀티모듈 빌드 스크립트에 흩어져 있던 공통 설정(compileSdk/minSdk, Java 17, Compose, Hilt 등)의 중복을 제거하기 위해 Now-in-Android 스타일 build-logic 컴포지트 빌드와 컨벤션 플러그인을 도입했습니다. 9개 모듈의 build.gradle.kts를 컨벤션 플러그인 기반으로 정리해 순 235줄을 줄였습니다.

작업 유형

  • feat: 새 기능
  • fix: 버그 수정
  • refactor: 리팩토링 (동작 변경 없음)
  • build: 빌드/의존성 설정
  • chore: 기타 (문서, 설정 등)
  • test: 테스트 추가/수정

변경 사항

  • build-logic 컴포지트 빌드 신설, 컨벤션 플러그인 6종 추가: gamss.android.application / gamss.android.library / gamss.android.compose / gamss.android.hilt / gamss.android.feature / gamss.jvm.library
  • 9개 모듈(app, domain, data, core:common, core:ui, feature:home/chat/calendar) 빌드 스크립트를 컨벤션 플러그인으로 이관
  • libs.versions.toml에 컨벤션 플러그인용 Gradle 플러그인 아티팩트 및 gamss-* 플러그인 alias 추가
  • settings.gradle.ktsincludeBuild("build-logic") 연결

관련 이슈

  • 해당 없음 (Notion으로 관리)

관련 작업 (Notion)

스크린샷 / 동작 화면

해당 없음 (빌드 설정 변경, 런타임/UI 동작 변경 없음)

체크리스트

  • base 브랜치가 develop으로 설정되어 있다
  • 로컬 빌드 및 실행을 확인했다 (assembleDebug, assembleRelease, test, detekt 모두 성공)
  • 커밋 메시지가 컨벤션(chore:, refactor:)을 따른다
  • 불필요한 로그/주석/디버그 코드를 제거했다
  • 리뷰어가 이해할 수 있도록 설명을 작성했다

리뷰 요청 사항

  • AGP 9.2.0에서 CommonExtension 제네릭 제거 및 defaultConfig/compileOptions/buildFeatures가 구체 확장으로 이동한 부분에 대응했습니다. 컨벤션 플러그인이 AGP 9 built-in Kotlin을 그대로 사용하도록 구성했으니 이 접근이 적절한지 봐주세요.
  • 각 모듈의 원본 설정과 이관 후 설정이 동등한지(특히 feature 3종의 공통 추출) 확인 부탁드립니다.

@soyeonLee126
soyeonLee126 requested a review from seunghee17 as a code owner July 23, 2026 02:54
@seunghee17

Copy link
Copy Markdown
Collaborator

feature 3종의 공통 플러그인·의존성·Android 설정이 기존과 동등하게 이관된 것을 확인했습니다. AGP 9 built-in Kotlin 및 구체 Extension 적용 방식도 적절하며 빌드도 정상 완료되어 merge 하겠습니다!

@seunghee17
seunghee17 merged commit 0f84014 into develop Jul 23, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants