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

fix(deps)!: update winit to 0.29 #21

Closed
wants to merge 2 commits into from

Conversation

tversteeg
Copy link

@tversteeg tversteeg commented Nov 4, 2023

My editor also applied cargo fmt.

It's a breaking change because a winit error is returned and because the winit versions are not compatible.

@tversteeg tversteeg changed the title chore(deps): update winit to 0.29 fix(deps)!: update winit to 0.29 Nov 4, 2023
@mkrasnitski
Copy link

There's a lot of extraneous formatting changes in this PR that would better be included in a separate PR. Regardless, this would be great to get merged as I'd like to use game-loop in combination with the newest version of winit.

src/helper.rs Outdated Show resolved Hide resolved
tuzz added a commit that referenced this pull request Feb 15, 2024
I decided not to apply the cargo fmt for now to keep the change clearer
in the commit history. Thanks @tversteeg for guiding me through the
update with your pull request:

#21
@tuzz
Copy link
Owner

tuzz commented Feb 15, 2024

Thanks @tversteeg, this change is now applied in version 1.1.0 of the crate. I decided not to apply the cargo fmt for now to keep the change clearer in the commit history. Closing now, thanks again.

@tuzz tuzz closed this Feb 15, 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.

3 participants