Skip to content

Conversation

madsmtm
Copy link
Member

@madsmtm madsmtm commented Sep 9, 2024

I believe I can keep the API as it currently is stable for a long time. The only real outstanding question is #24, which I haven't done enough research on to figure out (and isn't really motivated enough to do, it's quite niche), and I believe that can be solved without breaking changes.

The current API is even compatible with future versions of raw-window-handle that may use different handles.

@madsmtm madsmtm requested a review from MarijnS95 September 9, 2024 11:52
@madsmtm
Copy link
Member Author

madsmtm commented Sep 9, 2024

@MarijnS95 if you would review the API as it is currently, and tell me if you spot something odd, that'd be nice!

@MarijnS95
Copy link
Member

@madsmtm I think the only PR I've glossed over (which is probably the most important one) is #19 which I don't know too much about.

@madsmtm
Copy link
Member Author

madsmtm commented Sep 9, 2024

@madsmtm I think the only PR I've glossed over (which is probably the most important one) is #19 which I don't know too much about.

Yeah, that's fair enough, thanks for the review anyway!

@madsmtm madsmtm merged commit c38c759 into master Sep 9, 2024
4 checks passed
@madsmtm madsmtm deleted the new-release branch September 9, 2024 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants