[Chore /#124] ktLintCheck Configuration 추가 및 ktLint CI 삭제#126
Merged
Conversation
nahy-512
added
Fix 🐛
2 tasks
Contributor
📝 WalkthroughWalkthrough워크플로우 린트 체크 작업을 비활성화하고, ktLintCheck 실행을 위한 IDE 구성 파일을 추가하며, 빌드 그래들에서 주석 처리된 린트 설정을 제거하고, FlintSearchEmptyView 컴포저블에 기본 파라미터를 추가하는 변경사항입니다. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 3❌ Failed checks (3 warnings)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In
`@app/src/main/java/com/flint/core/designsystem/component/view/FlintSearchEmptyView.kt`:
- Around line 20-23: FlintSearchEmptyView의 title 기본값이 빈 문자열이라 호출부(예:
OnboardingFilmScreen)에서 인자를 생략하면 제목이 렌더링되지 않으니, FlintSearchEmptyView 함수 시그니처를
수정해 title을 nullable로 만들거나 빈값 체크를 추가하여 제목이 없을 때 Text를 렌더링하지 않도록 하세요; 예를 들어 변경할
대상은 FlintSearchEmptyView(title: String? = null) 또는 기존 title: String = "" 유지 후
Composable 내부에서 if (!title.isNullOrBlank()) Text(...)로 조건부 렌더링 처리하고, 필요하면
OnboardingFilmScreen 호출부에서 명시적으로 제목을 전달하도록 수정하세요.
🧹 Nitpick comments (3)
.github/workflows/pr_checker.yml (2)
1-1: 워크플로우 이름이 현재 기능과 맞지 않습니다.린트 작업이 비활성화되었으므로 워크플로우 이름을
Flint Build CI또는Flint PR Checker로 변경하는 것이 좋습니다.♻️ 제안된 수정
-name: Flint Lint CI +name: Flint Build CI
8-46: 주석 처리된 코드 삭제를 권장합니다.주석 처리된 린트 작업은 Git 히스토리에서 추적할 수 있으므로 삭제하는 것이 좋습니다. 나중에 다시 활성화할 가능성이 있다면 별도의 브랜치나 문서로 관리하거나, workflow_dispatch를 사용한 조건부 실행을 고려해 주세요.
♻️ 제안된 수정 - 주석 제거 후 간소화
name: Flint Build CI on: pull_request: branches: [ develop, main ] jobs: -# lint: -# name: PR Lint Check -# runs-on: ubuntu-latest -# -# steps: -# - name: Checkout -# uses: actions/checkout@v6 -# -# - name: Setup Gradle -# uses: gradle/actions/setup-gradle@v4 -# -# - name: Set up JDK 17 -# uses: actions/setup-java@v5 -# with: -# java-version: 17 -# distribution: 'temurin' -# -# - name: Set up Android SDK -# uses: android-actions/setup-android@v3 -# -# - name: Create local.properties -# env: -# DEV_BASE_URL: ${{ secrets.DEV_BASE_URL }} -# run: | -# echo "sdk.dir=$ANDROID_SDK_ROOT" > local.properties -# echo "base.url=\"${DEV_BASE_URL}\"" >> local.properties -# -# - name: Grant permission to gradlew -# run: chmod +x ./gradlew -# -# - name: Run ktlint -# run: ./gradlew ktlintCheck -# -# - name: Upload ktlint reports -# if: failure() -# uses: actions/upload-artifact@v4 -# with: -# name: ktlint-reports -# path: app/build/reports/ktlint/ build:app/src/main/java/com/flint/core/designsystem/component/view/FlintSearchEmptyView.kt (1)
37-41: 빈 문자열일 때 Text 숨김 처리 고려
title이 비어있을 때Text컴포저블을 숨기면 불필요한 렌더링과 레이아웃 공간 낭비를 방지할 수 있습니다.♻️ 조건부 렌더링 제안
Spacer(modifier = Modifier.height(12.dp)) - Text( - text = title, - color = FlintTheme.colors.white, - style = FlintTheme.typography.head3M18, - ) + if (title.isNotEmpty()) { + Text( + text = title, + color = FlintTheme.colors.white, + style = FlintTheme.typography.head3M18, + ) + }
Comment on lines
20
to
+23
| @Composable | ||
| fun FlintSearchEmptyView(title: String) { | ||
| fun FlintSearchEmptyView( | ||
| title: String = "" | ||
| ) { |
Contributor
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Check FlintSearchEmptyView implementation and usages
echo "=== FlintSearchEmptyView 파일 확인 ==="
find . -name "FlintSearchEmptyView.kt" -type f
echo ""
echo "=== FlintSearchEmptyView 호출 위치 및 사용 패턴 ==="
rg -n 'FlintSearchEmptyView' --type kotlin -A 3 -B 1Repository: imflint/Flint-Android
Length of output: 2790
🏁 Script executed:
cat -n app/src/main/java/com/flint/core/designsystem/component/view/FlintSearchEmptyView.ktRepository: imflint/Flint-Android
Length of output: 1962
title 파라미터 기본값이 공란으로 설정되어 있어 호출 시 제목이 누락되고 있습니다.
OnboardingFilmScreen의 호출부(153줄)에서 인자 없이 FlintSearchEmptyView()를 호출하고 있어, Text 컴포넌트가 빈 문자열로 렌더링됩니다. Preview(50줄)에서는 title = "아직 준비 중인 작품이에요"로 제목을 전달하는데, 실제 호출부에서는 제목이 누락되어 있습니다.
호출 측에서 적절한 제목을 명시적으로 전달하거나, 제목이 없을 경우 Text를 조건부로 렌더링하도록 수정해 주세요.
🤖 Prompt for AI Agents
In
`@app/src/main/java/com/flint/core/designsystem/component/view/FlintSearchEmptyView.kt`
around lines 20 - 23, FlintSearchEmptyView의 title 기본값이 빈 문자열이라 호출부(예:
OnboardingFilmScreen)에서 인자를 생략하면 제목이 렌더링되지 않으니, FlintSearchEmptyView 함수 시그니처를
수정해 title을 nullable로 만들거나 빈값 체크를 추가하여 제목이 없을 때 Text를 렌더링하지 않도록 하세요; 예를 들어 변경할
대상은 FlintSearchEmptyView(title: String? = null) 또는 기존 title: String = "" 유지 후
Composable 내부에서 if (!title.isNullOrBlank()) Text(...)로 조건부 렌더링 처리하고, 필요하면
OnboardingFilmScreen 호출부에서 명시적으로 제목을 전달하도록 수정하세요.
Contributor
Author
There was a problem hiding this comment.
의도한거야. hotfix라 나중에 고칠게
Contributor
There was a problem hiding this comment.
@nahy-512 알겠습니다! hotfix라면 나중에 수정하는 게 맞겠네요. 추후에 개선하실 때 참고해 주세요. 👍
🧠 Learnings used
Learnt from: kimjw2003
Repo: imflint/Flint-Android PR: 122
File: app/src/main/java/com/flint/presentation/splash/SplashScreen.kt:16-20
Timestamp: 2026-01-16T15:56:53.763Z
Learning: In the imflint/Flint-Android repository, do not flag or comment on FIXME comments, placeholder code, or test code with TODO/FIXME markers. The team prefers to handle these intentional placeholders on their own timeline without review comments.
Learnt from: ckals413
Repo: imflint/Flint-Android PR: 116
File: app/src/main/java/com/flint/presentation/onboarding/OnboardingDoneScreen.kt:97-105
Timestamp: 2026-01-16T08:47:13.979Z
Learning: In OnboardingDoneScreen.kt, the "시작하기" button is intentionally set to FlintButtonState.Disable as a placeholder state until ViewModel implementation is completed.
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
1 participant
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.
📮 관련 이슈
📌 작업 내용
🫛 To. 리뷰어
Summary by CodeRabbit
릴리스 노트
기능 개선
작업
✏️ Tip: You can customize this high-level summary in your review settings.