-
Notifications
You must be signed in to change notification settings - Fork 12
no_std support #57
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
no_std support #57
Conversation
Thanks! The MSRV is not specified, but I'm ok with setting one; would you like to include a version bump (to And please feel free to also look at #56 in case you see potential for any API changes. |
I just changed the base branch to the
|
@ljedrz I just applied the 2024 edition within my pr. Is it ready to merge into the 4.0 staging branch? |
It seems that bumping the edition requires some further changes; once the CI is green, I'm happy to merge into 4.0. |
@ljedrz Are you still looking into this? The CI currently requires approval. |
Apologies, I missed the notification. |
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.
LGTM!
Note that this raises the MSRV (if there is one). maybe version 4?