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

Bump windows crate range to 0.58-0.59 #72

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MarijnS95
Copy link

@MarijnS95 MarijnS95 commented Jan 8, 2025

https://github.com/microsoft/windows-rs/releases/tag/0.61.0

Because there were no breaking changes that affect this crate, the range could simply be extended to allow the new windows 0.59 release in addition to windows 0.58, allowing our MSRV to stay unmodified (windows 0.59 would have bumped it to 1.74 otherwise). However, such a should really be paired with a -Zminimal-versions or explicit downgrade in CI for testing though.

@MarijnS95
Copy link
Author

@Xudong-Huang is this something you can look into? wgpu is using this dependency :)

@MarijnS95
Copy link
Author

Also note that I haven't yet tested if this crate could take a version range on the windows crate, like I do in other crates that I maintain. Given that no code changes were necessary (though the CI does not appear to be running to confirm) both Windows versions should be compatible.

https://github.com/microsoft/windows-rs/releases/tag/0.61.0

Because there were no breaking changes that affect this crate, the
range could simply be extended to allow the new `windows 0.59` release
in addition to `windows 0.58`, allowing our MSRV to stay unmodified
(`windows 0.59` would have bumped it to `1.74` otherwise).  However,
such a should really be paired with a `-Zminimal-versions` or explicit
downgrade in CI for testing though.
@MarijnS95 MarijnS95 changed the title Bump windows crate to 0.59 Bump windows crate range to 0.58-0.59 Feb 8, 2025
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.

1 participant