Conversation
- Gradle 의존성 업데이트 및 빌드 설정 개선 - GitHub Actions 워크플로우 및 이슈 템플릿 추가 - 코드 스타일 가이드 및 문서화 파일 추가
✨ feat(core): 설정 파일 및 이슈 템플릿 추가
- 탭 타겟 코어 모듈의 핵심 기능 추가 및 예제 앱 구조 재구성 - 빌드 파일 및 설정 파일 수정으로 모듈 의존성 및 빌드 구성 개선 - 예제 앱의 UI 컴포넌트 및 리소스 이동 및 추가로 사용자 경험 향상
- TapTargetCoordinator 함수명 변경 및 Composable 함수로 변경 - TapTargetScope 클래스 추가 및 Modifier.tapTarget 확장 함수 리팩토링 - TapTargetDefinition 및 TapTargetStyle 데이터 클래스 추가 - TapTargetCoordinatorState 상태 관리 로직 개선
- iOS 앱 프로젝트 파일 및 빌드 설정 추가 - SwiftUI 기반 ContentView 및 iOSApp 컴포넌트 구현 - Xcode 프로젝트 및 구성 파일 추가로 iOS 지원 확장
- build.gradle.kts에 vanniktech maven publish 플러그인 적용 및 설정 추가 - libs.versions.toml 파일에서 의존성 버전一 및 최신화 - publish 관련 스크립트 및 Configuration 파일 삭제로 빌드 프로세스 간소화 - Android SDK 버전 및 Kotlin 버전 업데이트로 호환성 향상
- CLAUDE.md 파일 삭제 (-52 lines) - 프로젝트 문서 정리 및 구조 개선 - CMP 전환 관련 가이드 문서 제거
- README.md 파일의 구조와 내용을 정리하고 개선함 - 사용법과 예제 코드를 명확히 정리함 - 문서의 가독성과 정보 제공성을 높임
- android_example.gif와 ios_example.gif 이미지로 예시 변경 - tap-target-image.gif와 green-stash.gif 삭제 - README.md 파일에서 이미지 경로 및 테이블 구조 업데이트
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
이번 PR은 기존 Android 전용 라이브러리를 Compose Multiplatform(CMP)으로 전환하여 iOS 지원을 확장하고, 프로젝트 구조를 tap-target-cmp로 재편하며 배포 파이프라인을 현대화했습니다. 리뷰에서는 아직 정식 출시되지 않은 Kotlin 2.3.21 및 Gradle 9.1.0과 같은 불안정한 빌드 환경 설정을 안정 버전으로 되돌릴 것을 권고했습니다. 또한 패키지명 및 네임스페이스의 일관성 확보, 접근성을 위한 contentDescription 추가, 오타 수정, 그리고 주석 처리된 코드나 미사용 변수 등의 정리를 통해 코드 품질을 개선할 것을 제안했습니다. @ienground
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
- iosApp.xcscheme 파일의 빌드 및 실행 설정 구조 개선 - build.gradle.kts 파일에서 URL 및 namespace 수정 - Rendering.kt 파일의 컴포저블 함수 리팩토링 및 코드 정리
- MainViewController.kt 파일의 패키지 경로를 zone.ien.taptargetcmp.example로 이동 - 기존 패키지 경로 zone.ien.taptarget.cmp.example에서 변경 - ComposeUIViewController를 사용한 MainViewController 함수 정의 수정
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.
No description provided.