8000 Update dependencies by orowith2os · Pull Request #1303 · ggez/ggez · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Update dependencies #1303

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: devel
Choose a base branch
from
Open

Update dependencies #1303

wants to merge 1 commit into from

Conversation

orowith2os
Copy link

winit is untouched, see #1298 for updates on that

Please only open PRs that are based on and aimed towards the devel branch

For more information take a look at the contribution guide

@Bowarc
Copy link
Bowarc commented Jan 15, 2025

Is there any reason to not remove the toml dependency ?

@orowith2os
Copy link
Author

Not sure, also not really my focus here. But I can take another look in the morning.

@orowith2os
Update dependencies
5c90807 
winit is untouched, see ggez#1298 for
updates on that
@orowith2os
Copy link
Author

toml removed, now it only relies on toml_edit, and not both.

@a1phyr
Copy link
Member
a1phyr commented Jan 17, 2025

Could you use basic_toml instead ? I think that it is enough for our needs. Its API should look like toml 0.5. If it takes too much time, let it like that and I'll do it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@orowith2os @Bowarc @a1phyr
0