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

Handle generic classes when generating member injectors #30

Conversation

arctouch-ianribas
Copy link
Contributor

Generate correct member injectors for parameterized classes using the general star (*) for all type arguments of the target class.

Member injectors for non generic/parameterized classes works as before.

Fixes #29.

@outadoc
Copy link
Collaborator

outadoc commented Aug 21, 2024

@arctouch-ianribas Thank you for your contribution 🎉 I think it looks good.

Could you please rebase your PR onto the main branch?

Generate correct member injectors for parameterized classes using the general star (`*`) for all type arguments of the target class.

Member injectors for non generic/parameterized classes works as before.

Fixes BedrockStreaming#29.
@arctouch-ianribas arctouch-ianribas force-pushed the member-injector-for-parameterized-classes branch from 35518e8 to 2628290 Compare August 21, 2024 13:36
@arctouch-ianribas
Copy link
Contributor Author

Could you please rebase your PR onto the main branch?

Rebased.

Great to see the update to Kotlin 2.0 and KSP 2.

@outadoc
Copy link
Collaborator

outadoc commented Aug 22, 2024

@arctouch-ianribas Thanks again!

@outadoc outadoc merged commit 787d138 into BedrockStreaming:main Aug 22, 2024
1 check passed
@arctouch-ianribas arctouch-ianribas deleted the member-injector-for-parameterized-classes branch August 23, 2024 19:38
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.

Generated member injectors for parameterized classes don't compile
2 participants