Skip to content
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

Use ksp instead of kapt for Hilt #4418

Merged
merged 1 commit into from
Jun 2, 2024
Merged

Conversation

jpelgrom
Copy link
Member

Summary

When building the app with Kotlin 2.0, we get the following warning: w: Kapt currently doesn't support language version 2.0+. Falling back to 1.9.. This appears to be harmless, but considering kapt has been in maintenance mode for a couple of years and even Google samples are now using ksp for Hilt (where it is stable, even though Dagger itself still considers it 'alpha') this seems like a good time to migrate. After migrating Hilt there is nothing left using kapt so it is removed, and the warning resolved.

Did a build + run trying to hit most places where we inject things which seems to work fine. Not seeing any of the promised build speed improvements :(

Screenshots

n/a

Link to pull request in Documentation repository

n/a

Any other notes

@jpelgrom jpelgrom merged commit dff1fa3 into home-assistant:master Jun 2, 2024
4 checks passed
@jpelgrom jpelgrom deleted the hilt-ksp branch June 2, 2024 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants