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

Release v1.0 #25

Merged
merged 7 commits into from
Sep 9, 2024
Merged

Release v1.0 #25

merged 7 commits into from
Sep 9, 2024

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
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!

CHANGELOG.md Show resolved Hide resolved
Cargo.toml Outdated Show resolved Hide resolved
src/lib.rs Show resolved Hide resolved
@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