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

Add deps.rs badge to README #459

Merged
merged 1 commit into from
Feb 9, 2024
Merged

Conversation

uklotzde
Copy link
Contributor

@uklotzde uklotzde commented Feb 9, 2024

I have noticed that many dependencies are outdated. Upgrading them requires migrating existing code. Relying on unmaintained versions is not an option.

This PR adds a deps.rs badge to the README to show the current status.

@geom3trik
Copy link
Collaborator

Happy to add this badge, but just note that some of the code changes required to update these dependencies is quite substantial. For example, the winit update brought a lot of API breaking changes that we're still trying to work through.

@uklotzde
Copy link
Contributor Author

uklotzde commented Feb 9, 2024

...some of the code changes required to update these dependencies is quite substantial.

Sure. I noticed this too. All this does not need to be fixed immediately. But we should be aware of it.

Keeping pace with the building blocks is essential, especially during the early phase. Those breaking changes might even affect architectural decisions. Ignoring them for too long could be fatal is a risk.

@geom3trik geom3trik merged commit 34be918 into vizia:main Feb 9, 2024
9 checks passed
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.

None yet

2 participants