-
Notifications
You must be signed in to change notification settings - Fork 97
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
Updating the repo to latest edition #290
Updating the repo to latest edition #290
Conversation
@LucasLvy HF |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
jte hais
Addressed some comments, will continue later :) |
@LucasLvy trying to fix all those reaming struct pub fields but running in a lot of issues... |
i'd rather not, it's part of the migration, maybe use https://stacking.dev/? |
@LucasLvy Ready for round 2 |
STILL DRAFT WORKING ON IT
Pull Request type
Please check the type of change your PR introduces:
This PR updates the repository to use the latest edition.
This might have some breaking changes as some struct/fn/enums might now not be public by default but if any, it should be easy to identify and fix.
BREAKING some data serialization