-
Notifications
You must be signed in to change notification settings - Fork 2k
Upgrade kotlin and ksp dependencies. #5062
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
b7b40dc to
446f6ac
Compare
| kotlin = "2.3.0" | ||
| kotlinTarget = "1.9" | ||
| ksp = "2.2.20-2.0.3" | ||
| ksp = "2.4.3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be 2.3.4?
https://github.com/google/ksp/releases
| kotlin = "2.3.0" | ||
| kotlinTarget = "1.9" | ||
| ksp = "2.2.20-2.0.3" | ||
| ksp = "2.4.3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| ksp = "2.4.3" | |
| ksp = "2.3.4" |
446f6ac to
0a770ff
Compare
|
Hi team, We're currently blocked on upgrading to Kotlin 2.3.0 due to this As other reviewers mentioned, the KSP version should be 2.3.4 Is there a timeline for merging this fix and releasing a new version? Thanks for your work on Dagger/Hilt! |
You should not be blocked by this. See the comments in #5001 for the workaround.
The KSP version (while incorrect) is not the reason this PR has not been merged. As mentioned in #5001 (comment), we're waiting on our testing infra to upgrade to Kotlin 2.3.0. |
Upgrade kotlin and ksp dependencies.
Fixes #5001
Fixes #5059
RELNOTES=Fixes #5001, #5059: Upgrade kotlin and ksp dependencies.