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

Kotlin classes can inherit from traits. #2297

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

mhammond
Copy link
Member

@mhammond mhammond commented Nov 1, 2024

Builds on #2204 which landed the metadata and Python support.

@mhammond mhammond requested a review from bendk November 1, 2024 02:42
@mhammond mhammond requested a review from a team as a code owner November 1, 2024 02:42
Copy link
Contributor

@bendk bendk left a comment

Choose a reason for hiding this comment

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

Nice, this looks great. I'm not sure why that keywords fixture is failing, maybe something needs to be quoted?

Builds on mozilla#2204 which landed the metadata and Python support.

Also fixes minor issues with `CallbackInterface` types.
@mhammond
Copy link
Member Author

mhammond commented Nov 7, 2024

turned out to be a few minor issues with callback interfaces. Then I discovered the trait and struct for that fixture both had the same method with different signatures 🙃 That's an interesting edge-case for the future, but I just avoided doing that for now. Fixed all that and a couple of other tweaks but all of which were sane and small enough that I didn't think it needed a new review.

@mhammond mhammond merged commit b7b3880 into mozilla:main Nov 7, 2024
5 checks passed
@mhammond mhammond deleted the push-vptutpmlxunr branch November 7, 2024 04:04
mhammond added a commit to mhammond/uniffi-rs that referenced this pull request Dec 22, 2024
mhammond added a commit to mhammond/uniffi-rs that referenced this pull request Dec 22, 2024
mhammond added a commit that referenced this pull request Dec 24, 2024
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.

2 participants