Feature/#7 custom plugin #9

ashwon12 · 2025-01-10T05:38:07Z

💡 Issue

Resolved: [Feature] Convention Plugin 로 공통 빌드구성 #7

🌱 Key changes

커스텀 컨밴션 plugin적용으로 공통 빌드 사항 통합

✅ To Reviewers

컨밴션 관련

생성한 컨밴션 : application / android / kotlin / compose 사용 하는 곳 / feature 모듈 공통사항
가장 작은 단위들로만 만들었어요 ! 개발하다가 공통 되는 항목들이 있으면 하나씩 덧붙여도 될 것 같습니다.
테스트 관련된 컨밴션은 어떻게 해야할지 판단이 안서서 우선은 생략했습니다...
app 모듈에서 compose 적용을 제외해서 화면을 제거해 놓은 상태입니닷 ( hilt 적용해서 화면 보이도록 후딱 수정하겠습니닷 ,, 후딱 .. )
libs.versions.toml 파일은 개발하면서 정리해가는것이 덜 헷갈릴것 같아요 !!!

변경사항

java 17로 올렸어요 ~
중간에 빌드가 안되어서 agp 버전을 낮췄는데 이후에 다시 업그레이드 검토하겠습니다.

처음 적용해 보는 거라 누락되거나 적절하지 않은 사용이 있을 것 같아요,, !!!!! 확인 부탁드리겠습니다
approve 되면 디자인시스템 머지 후에 리베이스 머지하도록 하겠습니다

비고

참고 프로젝트 : Now in Android , DroidKnightsApp

Summary by CodeRabbit

빌드 구성 변경
- Gradle 빌드 로직 간소화
- 모듈 네임스페이스 재구성
- 플러그인 및 의존성 최적화
프로젝트 구조 변경
- 앱 모듈 제거
- 테마 및 컬러 관련 파일 삭제
- 메인 액티비티 제거
의존성 관리
- 코어 모듈 의존성 재정의
- 외부 라이브러리 의존성 감소
- 컴포즈 관련 의존성 업데이트
개발 환경
- Kotlin DSL 빌드 스크립트 도입
- 빌드 로직 모듈 추가
- 버전 카탈로그 개선

coderabbitai · 2025-01-10T05:38:13Z

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

개요

워크스루

이 풀 리퀘스트는 빌드 로직 모듈을 도입하고 프로젝트의 빌드 구성을 표준화하는 대규모 리팩토링을 포함합니다. 주요 변경 사항은 공통 빌드 플러그인 생성, 네임스페이스 설정 함수 추가, 여러 모듈의 빌드 스크립트 단순화, 그리고 종속성 관리 방식 변경을 포함합니다.

변경 사항

파일/디렉토리	변경 요약
`build-logic/`	새로운 빌드 로직 모듈 추가, 공통 빌드 플러그인 및 확장 함수 생성
`app/`, `core/`, `feature/` 모듈의 `build.gradle.kts`	빌드 스크립트 단순화, 네임스페이스 설정 함수 적용, 플러그인 및 종속성 감소
`gradle/libs.versions.toml`	라이브러리 버전 업데이트 및 새로운 그레이들 플러그인 추가
`settings.gradle.kts`	`build-logic` 모듈 포함 및 `:app` 모듈 제거

평가 대상 이슈

목표	해결 여부	설명
Convention Plugin로 공통 빌드 구성	✅	빌드 로직 모듈 생성 및 공통 빌드 구성 적용

시퀀스 다이어그램

sequenceDiagram
    participant BuildLogic as Build Logic Module
    participant Gradle as Gradle Build System
    participant Modules as Project Modules

    BuildLogic->>Gradle: Apply Convention Plugins
    Gradle->>Modules: Configure Build Settings
    Modules->>BuildLogic: Use Shared Configuration
    BuildLogic->>Gradle: Set Namespace
    BuildLogic->>Gradle: Manage Dependencies

시 (토끼의 빌드 리팩토링 축하시)

🐰 빌드 스크립트여, 춤추어라
플러그인 손에 들고서
네임스페이스 간결하게
의존성 가볍게 만들어
토끼의 코드, 경쾌하게! 🎉

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

‼️ IMPORTANT
Auto-reply has been disabled for this repository in the CodeRabbit settings. The CodeRabbit bot will not respond to your replies unless it is explicitly tagged.

Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
- @coderabbitai generate unit testing code for this file.
- @coderabbitai modularize this function.
PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
- @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
- @coderabbitai read src/utils.ts and generate unit testing code.
- @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
- @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

@coderabbitai pause to pause the reviews on a PR.
@coderabbitai resume to resume the paused reviews.
@coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
@coderabbitai full review to do a full review from scratch and review all the files again.
@coderabbitai summary to regenerate the summary of the PR.
@coderabbitai generate docstrings to generate docstrings for this PR. (Beta)
@coderabbitai resolve resolve all the CodeRabbit review comments.
@coderabbitai configuration to show the current CodeRabbit configuration for the repository.
@coderabbitai help to get help.

Other keywords and placeholders

Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (`.coderabbit.yaml`)

You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
Please see the configuration documentation for more information.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

Visit our Documentation for detailed information on how to use CodeRabbit.
Join our Discord Community to get help, request features, and share feedback.
Follow us on X/Twitter for updates and announcements.

jinukeu

오! 고생하셨습니다!!
제가 예전에 했던 방식에서 뭔가 더 새로운게 추가된거같네요. 다음에 천천히 보면서 좀 배우겠습니다 ㅎㅎ

minor한 코멘트만 있어서 바로 merge해도 될거같아요 (세팅 부분이라 코드 리뷰할게 크게 없네요)

힐트까지 기대하겠습니다!

jinukeu · 2025-01-10T05:42:11Z

build-logic/src/main/java/com/yapp/app/AppNameExtension.kt

+
+fun Project.setNamespace(name: String) {
+    androidExtension.apply {
+        namespace = "com.yapp.app.$name"


오옹 이건 어디서 참고하신건가요?

드로이드 나이츠 참고했습니다 ~
name을 컨밴션으로 관리하고 있더라구영

오 신기하네요 하나 배워갑니다 👍

jinukeu · 2025-01-10T05:43:10Z

gradle/libs.versions.toml

@@ -1,18 +1,22 @@
 [versions]
-agp = "8.7.3"
+agp = "8.3.2"


버전 내리신 이유가 있나요?

초반 빌드 구성할때 kotlin 버전이랑 안맞다는 오류가 떠서 낮춘상태로 진행했는데 ,, 지금은 올려도 잘 되네용 ~
꽤 많이 낮췄어서 다시 버전 업 시키는게 좋을것 같네요 !

jinukeu · 2025-01-10T05:43:30Z

gradle/libs.versions.toml

-activityCompose = "1.8.0"
-composeBom = "2024.04.01"
+activityCompose = "1.9.0"
+composeBom = "2024.05.00"


24.05가 최신이던가용?

컨밴션만 세우고 버전은 전체적으로 체크를 안한 상태였어서 ..
다음 pr때 한번에 체크 한번 쫙 해서 반영하도록 할게요 ~~~~

coderabbitai · 2025-01-10T06:01:14Z

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

coderabbitai

Actionable comments posted: 14

📜 Review details

Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 42ceb74 and 73ed3c2.

📒 Files selected for processing (29)

app/build.gradle.kts (2 hunks)
app/src/main/java/com/yapp/official/MainActivity.kt (0 hunks)
app/src/main/java/com/yapp/official/ui/theme/Color.kt (0 hunks)
app/src/main/java/com/yapp/official/ui/theme/Theme.kt (0 hunks)
app/src/main/java/com/yapp/official/ui/theme/Type.kt (0 hunks)
build-logic/.gitignore (1 hunks)
build-logic/build.gradle.kts (1 hunks)
build-logic/settings.gradle.kts (1 hunks)
build-logic/src/main/java/com/yapp/app/AppNameExtension.kt (1 hunks)
build-logic/src/main/java/com/yapp/app/ComposeAndroid.kt (1 hunks)
build-logic/src/main/java/com/yapp/app/Extension.kt (1 hunks)
build-logic/src/main/java/com/yapp/app/KotlinAndroid.kt (1 hunks)
build-logic/src/main/java/yapp.android.application.gradle.kts (1 hunks)
build-logic/src/main/java/yapp.android.compose.gradle.kts (1 hunks)
build-logic/src/main/java/yapp.android.feature.gradle.kts (1 hunks)
build-logic/src/main/java/yapp.android.library.gradle.kts (1 hunks)
build-logic/src/main/java/yapp.kotlin.library.gradle.kts (1 hunks)
core/data-api/build.gradle.kts (1 hunks)
core/data/build.gradle.kts (1 hunks)
core/designsystem/build.gradle.kts (1 hunks)
core/domain/build.gradle.kts (1 hunks)
core/model/build.gradle.kts (1 hunks)
core/ui/build.gradle.kts (1 hunks)
feature/home/build.gradle.kts (1 hunks)
feature/login/build.gradle.kts (1 hunks)
feature/notice/build.gradle.kts (1 hunks)
feature/signup/build.gradle.kts (1 hunks)
gradle/libs.versions.toml (1 hunks)
settings.gradle.kts (2 hunks)

💤 Files with no reviewable changes (4)

app/src/main/java/com/yapp/official/ui/theme/Type.kt
app/src/main/java/com/yapp/official/MainActivity.kt
app/src/main/java/com/yapp/official/ui/theme/Color.kt
app/src/main/java/com/yapp/official/ui/theme/Theme.kt

🔇 Additional comments (20)

build-logic/.gitignore (1)

1-1: 빌드 디렉토리 제외 설정이 적절합니다!

빌드 산출물을 버전 관리에서 제외하는 것은 올바른 관행입니다.
core/model/build.gradle.kts (1)

2-2: 커스텀 플러그인 적용 확인이 필요합니다.

yapp.kotlin.library 플러그인으로 전환하면서 Java 호환성 설정이 제거되었습니다. 다음 사항들을 확인해주세요:

Java 17 업그레이드가 모든 모듈에서 정상적으로 적용되었는지

기존 Java 라이브러리 의존성들과의 호환성

✅ Verification successful

Java 17 설정이 커스텀 플러그인을 통해 올바르게 적용되었습니다.

yapp.kotlin.library 플러그인이 core 모듈들에 일관되게 적용되어 있습니다.

플러그인에서 Java 17과 Kotlin JVM 17 타겟이 올바르게 설정되어 있습니다.

다른 모듈에서 충돌되는 버전 설정이 발견되지 않았습니다.
🏁 Scripts executed

The following scripts were executed for the analysis:

Script:
#!/bin/bash
# Java 버전 설정 확인
rg "JavaVersion" -A 2
# Kotlin JVM 타겟 설정 확인
rg "jvmTarget" -A 2
Length of output: 2179

Script:
#!/bin/bash
# 모든 모듈의 build.gradle.kts 파일에서 플러그인 적용 확인
echo "=== Checking plugin application ==="
fd build.gradle.kts --type f --exec rg "id$\"yapp\.kotlin\.library\"$" {} -l

echo -e "\n=== Checking other Java/Kotlin version settings ==="
# 다른 Java/Kotlin 버전 설정이 있는지 확인
fd build.gradle.kts --type f --exec rg "(sourceCompatibility|targetCompatibility|jvmTarget)" {} -l
Length of output: 446
core/data-api/build.gradle.kts (1)

2-2: 일관된 플러그인 구성이 적용되었습니다!

model 모듈과 동일한 방식으로 커스텀 플러그인이 적용되어 일관성이 유지되었습니다.

build-logic/src/main/java/yapp.android.library.gradle.kts (1)

1-7: 모듈 구성이 깔끔하고 재사용 가능하게 잘 설계되었습니다!

안드로이드 라이브러리 모듈을 위한 기본 설정이 잘 구성되어 있으며, Kotlin 설정을 별도의 함수로 분리한 것이 좋은 접근 방식입니다.
build-logic/build.gradle.kts (1)

7-9: 버전 관리 방식 개선을 제안드립니다.

Gradle 플러그인 버전들을 libs.versions.toml 파일에서 중앙 관리하고 있는지 확인이 필요합니다. 이는 버전 충돌과 업데이트를 효과적으로 관리하는데 도움이 됩니다.

다음 명령어로 버전 카탈로그 설정을 확인해보세요:

✅ Verification successful

버전 관리가 적절히 구현되어 있습니다.

libs.versions.toml 파일에서 모든 Gradle 플러그인 버전이 중앙 집중식으로 관리되고 있음을 확인했습니다. Android Gradle Plugin, Kotlin Gradle Plugin, Compose Compiler Plugin 모두 버전 카탈로그를 통해 적절히 관리되고 있습니다.
🏁 Scripts executed

The following scripts were executed for the analysis:

Script:
#!/bin/bash
echo "Checking version catalog configuration..."
cat gradle/libs.versions.toml 2>/dev/null || echo "libs.versions.toml not found"
Length of output: 2657
core/ui/build.gradle.kts (1)

13-16: UI 모듈에 필요한 의존성이 누락되었을 수 있습니다.

UI 모듈임에도 불구하고 최소한의 의존성만 포함되어 있습니다. 다음 라이브러리들의 필요 여부를 검토해주세요:

androidx.core.ktx

androidx.lifecycle

material3

현재 모듈에서 사용 중인 Android UI 컴포넌트를 확인하기 위해 다음 스크립트를 실행해보세요:

feature/home/build.gradle.kts (1)

11-15: 의존성 정리 검토

코어 모듈 의존성들이 제거되었습니다. 이는 yapp.android.feature 플러그인에서 공통 의존성으로 관리되는 것으로 보이나, 다음을 확인해주세요:

제거된 의존성들이 플러그인에 포함되어 있는지 확인

테스트 관련 의존성 제거가 의도된 것인지 확인
feature/login/build.gradle.kts (1)

1-15: 모듈 간 일관성 확인

feature/home 모듈과 동일한 패턴으로 변경되었습니다. 일관성 있는 구조는 좋으나, 다음을 확인해주세요:

로그인 기능에 특화된 추가 설정이나 의존성이 필요하지는 않은지 검토

기존 com.yapp.login 패키지를 사용하는 다른 모듈들의 참조 업데이트 필요

✅ Verification successful

패키지명 변경 안전성 확인 완료

패키지명 변경이 다른 모듈에 영향을 주지 않음을 확인했습니다. 다만 다음 작업이 필요합니다:

ExampleUnitTest.kt와 ExampleInstrumentedTest.kt의 패키지명을 feature.login으로 업데이트 필요
🏁 Scripts executed

The following scripts were executed for the analysis:

Script:
#!/bin/bash
# 로그인 관련 특수 의존성 확인
rg "com\.yapp\.login" 
Length of output: 353

Script:
#!/bin/bash
# Check for project dependencies referencing login module
fd build.gradle.kts --exec grep -l "login" {} \;

# Check for imports from login module in other modules
rg "^import.*login" --type kotlin
Length of output: 143

Script:
#!/bin/bash
# Check how login module is referenced in app/build.gradle.kts
cat app/build.gradle.kts
Length of output: 579
feature/notice/build.gradle.kts (1)

1-15: 일관된 구조 적용 확인

다른 feature 모듈들과 동일한 구조로 변경되었습니다. 주의할 점:

공통 플러그인으로 이동된 설정들이 notice 모듈의 요구사항을 모두 충족하는지 확인

네임스페이스 변경(com.yapp.notice → feature.notice)에 따른 리팩토링 필요

feature/signup/build.gradle.kts (1)

1-15: 회원가입 모듈 설정 검토

다른 feature 모듈들과 동일한 패턴을 따르고 있습니다. 확인사항:

회원가입 프로세스에 필요한 특수 설정이나 의존성이 플러그인에서 누락되지 않았는지 검토

패키지명 변경으로 인한 참조 업데이트 필요

core/domain/build.gradle.kts (2)

4-4: 커스텀 플러그인 적용이 적절합니다.

기존의 플러그인 별칭 대신 커스텀 컨벤션 플러그인을 사용하여 빌드 구성을 통합하는 것이 좋은 접근 방식입니다.

8-8: 네임스페이스 설정 방식이 개선되었습니다.

setNamespace 확장 함수를 사용하여 일관된 방식으로 네임스페이스를 관리하는 것이 좋습니다.

build-logic/settings.gradle.kts (2)

1-2: 타입 안전한 프로젝트 접근자 활성화가 적절합니다.

TYPESAFE_PROJECT_ACCESSORS 기능을 활성화하여 프로젝트 간 의존성 참조의 안정성을 향상시켰습니다.

8-11: 버전 카탈로그 설정이 적절합니다.

공통 버전 관리를 위해 libs.versions.toml 파일을 사용하는 것이 좋은 접근 방식입니다. PR 목표에서 언급된 대로 점진적으로 구성해 나가는 것이 혼란을 줄일 수 있습니다.

build-logic/src/main/java/yapp.android.feature.gradle.kts (2)

1-4: 플러그인 구성이 적절합니다.

feature 모듈을 위한 기본 플러그인 구성이 잘 되어있습니다. 라이브러리와 Compose 플러그인을 함께 적용하여 UI 구현에 필요한 기반을 제공합니다.

14-19: 핵심 모듈 의존성 검증이 필요합니다.

feature 모듈의 기본 의존성으로 설정된 core 모듈들이 모든 feature에서 실제로 필요한지 확인이 필요합니다.

app/build.gradle.kts (1)

21-23: 의존성 정리 계획을 구체화해야 합니다.

Compose 및 Android 라이브러리 제거 계획이 있는데, 제거 순서와 영향도를 미리 분석하는 것이 좋습니다.

settings.gradle.kts (2)

2-2: 빌드 로직 모듈 포함이 적절합니다.

커스텀 컨벤션 플러그인을 위한 build-logic 모듈 포함이 적절합니다. 빌드 구성의 재사용성과 유지보수성이 향상될 것으로 기대됩니다.

Line range hint 4-12: 저장소 필터링 구성이 적절합니다.

Google 저장소의 콘텐츠 필터링이 잘 설정되어 있습니다. 필요한 그룹만 포함하여 의존성 해결 과정의 효율성을 높였습니다.
gradle/libs.versions.toml (1)

16-19: Gradle 플러그인 버전 업그레이드 검토 필요

이전 리뷰에서 논의된 것처럼, 현재 낮은 버전의 AGP를 사용하고 있습니다. 빌드 안정성을 위해 다음 사항들을 검토해주세요:

AGP 버전 업그레이드 가능성

Kotlin 버전과의 호환성

다른 Gradle 플러그인들과의 호환성

다음 스크립트를 실행하여 최신 안정 버전들을 확인해보세요:
#!/bin/bash
# Description: Check latest stable versions of AGP and Kotlin

echo "Latest AGP version:"
curl -s https://maven.google.com/web/index.html | grep -o 'com.android.tools.build:gradle:[0-9]*\.[0-9]*\.[0-9]*' | sort -V | tail -n 1

echo -e "\nLatest Kotlin version:"
curl -s https://repo1.maven.org/maven2/org/jetbrains/kotlin/kotlin-gradle-plugin/maven-metadata.xml | grep -o '<version>[0-9]*\.[0-9]*\.[0-9]*</version>' | sort -V | tail -n 1

build-logic/src/main/java/com/yapp/app/AppNameExtension.kt

build-logic/src/main/java/com/yapp/app/Extension.kt

coderabbitai · 2025-01-10T06:06:54Z

build-logic/src/main/java/com/yapp/app/ComposeAndroid.kt

+        dependencies {
+            val bom = libs.findLibrary("androidx-compose-bom").get()
+            add("implementation", platform(bom))
+            add("androidTestImplementation", platform(bom))
+
+            add("implementation", libs.findLibrary("androidx.material3").get())
+            add("implementation", libs.findLibrary("androidx.ui").get())
+            add("implementation", libs.findLibrary("androidx.ui.tooling.preview").get())
+            add("androidTestImplementation", libs.findLibrary("androidx.junit").get())
+            add("androidTestImplementation", libs.findLibrary("androidx.espresso.core").get())
+            add("androidTestImplementation", libs.findLibrary("androidx.ui.test.junit4").get())
+            add("debugImplementation", libs.findLibrary("androidx.ui.tooling").get())
+            add("debugImplementation", libs.findLibrary("androidx.ui.test.manifest").get())
+        }


🛠️ Refactor suggestion

의존성 검색 실패에 대한 예외 처리 필요

findLibrary().get()을 직접 호출하면 의존성을 찾지 못했을 때 예외가 발생할 수 있습니다.

다음과 같이 수정하는 것을 제안합니다:

dependencies { - val bom = libs.findLibrary("androidx-compose-bom").get() + val bom = libs.findLibrary("androidx-compose-bom").getOrNull() + ?: error("androidx-compose-bom 의존성을 찾을 수 없습니다. libs.versions.toml 파일을 확인해주세요.") add("implementation", platform(bom)) // ... 다른 의존성들에도 동일한 패턴 적용 }

Committable suggestion skipped: line range outside the PR's diff.

coderabbitai · 2025-01-10T06:06:54Z

build-logic/src/main/java/com/yapp/app/KotlinAndroid.kt

+            val warningsAsErrors: String? by project
+            allWarningsAsErrors.set(warningsAsErrors.toBoolean())


🧹 Nitpick (assertive)

warningsAsErrors 속성의 기본값 설정 필요

warningsAsErrors 프로젝트 속성이 없을 경우의 기본값 처리가 필요합니다.

다음과 같이 수정하는 것을 제안합니다:

- val warningsAsErrors: String? by project - allWarningsAsErrors.set(warningsAsErrors.toBoolean()) + val warningsAsErrors: String? by project + allWarningsAsErrors.set(warningsAsErrors?.toBoolean() ?: false)

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

val warningsAsErrors: String? by project

allWarningsAsErrors.set(warningsAsErrors.toBoolean())

val warningsAsErrors: String? by project

allWarningsAsErrors.set(warningsAsErrors?.toBoolean() ?: false)

@coderabbitai 속성이 없을 경우가 있을까 .... ?

@ashwon12 네, 속성이 없는 경우가 있을 수 있습니다. warningsAsErrors 속성은 다음과 같은 방법으로 설정될 수 있습니다:

gradle.properties 파일에서 설정

명령줄 인자로 설정 (-PwarningsAsErrors=true)

환경 변수로 설정

이러한 방법으로 설정되지 않은 경우 속성은 null이 됩니다. 일반적으로 warnings as errors는 CI/CD 환경이나 특정 빌드 설정에서만 활성화하는 것이 일반적이므로, 기본값을 false로 설정하는 것이 안전합니다.

✏️ Learnings added

Learnt from: ashwon12 PR: YAPP-admin/yappu-world-android#9 File: build-logic/src/main/java/com/yapp/app/KotlinAndroid.kt:72-73 Timestamp: 2025-01-10T06:16:27.206Z Learning: In Kotlin Android projects, the `warningsAsErrors` property controls whether compiler warnings should be treated as errors. This property is typically disabled by default and only enabled in specific environments like CI/CD.

Note: Learnings are effective only in the context of similar code segments. To apply general coding guidelines, please configure review instructions. You can manage existing learnings in the app.

build-logic/src/main/java/com/yapp/app/KotlinAndroid.kt

core/designsystem/build.gradle.kts

feature/home/build.gradle.kts

core/data/build.gradle.kts

app/build.gradle.kts

gradle/libs.versions.toml

ashwon12 added the Feature 기능 추가, 개발 label Jan 10, 2025

ashwon12 requested a review from jinukeu January 10, 2025 05:38

ashwon12 self-assigned this Jan 10, 2025

jinukeu approved these changes Jan 10, 2025

View reviewed changes

ashwon12 added the CodeRabbit CodeRabbit 코드 리뷰 요청 label Jan 10, 2025

coderabbitai bot reviewed Jan 10, 2025

View reviewed changes

ashwon12 added 9 commits January 11, 2025 02:53

feat/#7 : builg-login 모듈 생성

52e7e79

feat/#7 : kts 초기 설정 ( app 기본 컨밴션 적용 )

1422547

feat/#7 : feature 컨밴션 생성

3bee98f

feat/#7 : feature 모듈 공통 빌드 적용

fcb5ef9

feat/#7 : core모듈 공통 빌드 및 의존성 설정

891aea7

feat/#7 : AppNameExtension 생성

8346c41

feat/#7 : agp 버전 업

bbe37e9

feat/#7 : 네임스페이스와 패키지 경로 일치

0b7e178

feat/#7 : namespace 설정으로 인한 패키지 경로 일치하도록 수정

4f7d02e

ashwon12 force-pushed the feature/#7-custom_plugin branch from d1944f5 to 4f7d02e Compare January 10, 2025 18:00

ashwon12 merged commit 5fd6341 into develop Jan 10, 2025

ashwon12 deleted the feature/#7-custom_plugin branch January 10, 2025 18:02

coderabbitai bot mentioned this pull request Jan 21, 2025

Feature/#26 navigation #27

Merged

ashwon12 added a commit that referenced this pull request Apr 7, 2025

fix: #9 회원가입 대기중인 계정 로그인 시도시 대기중 화면으로 이동

afb5651

ashwon12 added a commit that referenced this pull request Apr 8, 2025

fix: #9 회원가입 대기중인 계정 로그인 시도시 대기중 화면으로 이동

90ec877

ashwon12 added a commit that referenced this pull request Apr 8, 2025

fix: #9 회원가입 대기중인 계정 로그인 시도시 대기중 화면으로 이동

0c21a90

		val warningsAsErrors: String? by project
		allWarningsAsErrors.set(warningsAsErrors.toBoolean())

Feature/#7 custom plugin #9

Feature/#7 custom plugin #9

Uh oh!

Conversation

ashwon12 commented Jan 10, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

💡 Issue

🌱 Key changes

✅ To Reviewers

컨밴션 관련

변경사항

비고

Summary by CodeRabbit

Uh oh!

coderabbitai bot commented Jan 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

개요

워크스루

변경 사항

평가 대상 이슈

관련될 수 있는 PR

시퀀스 다이어그램

시 (토끼의 빌드 리팩토링 축하시)

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (.coderabbit.yaml)

Documentation and Community

Uh oh!

jinukeu left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot commented Jan 10, 2025

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

coderabbitai bot Jan 10, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot Jan 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot Jan 10, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ashwon12 commented Jan 10, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Jan 10, 2025 •

edited

Loading

CodeRabbit Configuration File (`.coderabbit.yaml`)

coderabbitai bot Jan 10, 2025 •

edited

Loading