Skip to content

Conversation

@PeraSite
Copy link
Member

Summary

kapt 컴파일 관련 오류(관련 YouTrack 링크)를 해결하기 위해 코틀린 버전을 1.9.25로 업그레이드

Describe your changes

To reviewers

  • 마이너 버전 업그레이드긴 하나, 언어의 버전업은 항상 조심해야하므로 테스트가 필요합니다.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Summary of Changes

Hello @PeraSite, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 JDK 17 이상 환경에서 발생하는 kapt 컴파일 오류를 해결하기 위해 Kotlin 버전을 업데이트합니다. 이 변경은 빌드 안정성을 확보하고 최신 개발 환경과의 호환성을 개선하는 데 중점을 둡니다.

Highlights

  • Kotlin 버전 업그레이드: kotlin-android 패키지 버전을 1.9.0에서 1.9.25로 업데이트하여 JDK 17+ 환경에서의 빌드 문제를 해결합니다.
  • Compose 호환성 업데이트: Kotlin 버전 업그레이드에 맞춰 kotlinCompilerExtensionVersion을 1.5.0에서 1.5.15로 조정하여 Compose와의 호환성을 유지합니다.
  • 빌드 오류 해결: JDK 17 이상에서 발생하는 kapt 컴파일 관련 IllegalAccessError를 해결합니다.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in issue comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

안녕하세요. JDK 17 이상에서 발생하는 빌드 문제를 해결하기 위해 코틀린 버전을 업그레이드하는 PR을 작성해주셔서 감사합니다.

YouTrack 이슈를 해결하기 위해 코틀린 버전을 1.9.25로 올리고, 관련하여 Compose Compiler 버전도 함께 수정해주신 점이 좋습니다. 이를 통해 최신 개발 환경에 대응할 수 있게 되었습니다.

다만, 한 가지 확인이 필요한 부분이 있습니다. kotlinCompilerExtensionVersion1.5.15로 설정하셨는데, 이 버전은 코틀린 2.0.0과 호환되는 버전입니다. 현재 프로젝트의 코틀린 버전인 1.9.25와는 호환성이 보장되지 않을 수 있어 잠재적인 문제를 야기할 수 있습니다. 관련하여 상세한 리뷰 코멘트를 남겼으니 확인 부탁드립니다.

전반적으로 문제 해결을 위한 좋은 시도이며, 제안 드린 부분을 수정하면 더 안정적인 프로젝트가 될 것 같습니다.

@kangyuri1114
Copy link
Member

이전 pr (새로운 피처 개발 브랜치)를 디벨롭에 머지 후 여기 브랜치에서 리베이스 한 다음.. 테스트 하고 문제 없으면 반영하는 것이 좋을 듯 합니다

@PeraSite
Copy link
Member Author

디벨롭 브랜치에서 새로운 브랜치를 만들 때 이 PR이 머지되지 않으면 항상 빌드 오류가 떠서 빠르게 디벨롭 브랜치에 머지하는게 좋긴 합니다.
다른 피처 브랜치에서 코틀린 버전업을 이미 해서 충돌이 일어나는 경우가 있을까요?

Copy link
Member

@HI-JIN2 HI-JIN2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#316 pr에서 버전 업 하지 않은 것을 확인했습니다. 제가 #251 pr에서 수정하긴 했는데, 제가 인지 하였으니 괜찮을 것 같습니다. CC @kangyuri1114

+) Pr 제목에 prefix는 첫글자만 대문자로 해주세요! [Fix]

@PeraSite PeraSite changed the title [FIX] JDK 17 이상에서 발생하는 빌드 문제 코틀린 버전업으로 해결 [Fix] JDK 17 이상에서 발생하는 빌드 문제 코틀린 버전업으로 해결 Aug 28, 2025
@PeraSite PeraSite merged commit 5053607 into develop Aug 28, 2025
1 check passed
@PeraSite PeraSite deleted the fix/kotlin-version branch August 28, 2025 16:20
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.

4 participants