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

Revert veldrid update #5929

Merged
merged 2 commits into from
Jul 13, 2023
Merged

Revert veldrid update #5929

merged 2 commits into from
Jul 13, 2023

Conversation

This reverts commit 02626fd.
@bdach
Copy link
Collaborator

bdach commented Jul 13, 2023

OP is confusing, I'm pretty sure it should say that it's reverting ppy/veldrid#30 and ppy/veldrid#31? (it currently says it's reverting ppy/veldrid#32, which is a revert, so 🤯)

Other than that, looks good in theory. Someone with a Mac should probably confirm in practice, though.

@peppy
Copy link
Member Author

peppy commented Jul 13, 2023

Yeah, misread the PR title. Fixed now.

bdach
bdach previously approved these changes Jul 13, 2023
@frenzibyte frenzibyte merged commit 2bda3a1 into ppy:master Jul 13, 2023
@peppy peppy deleted the unbump-veldrid branch September 14, 2023 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants