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 support for no_std #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

daxpedda
Copy link

This adds support to no_std environments by using default-features = false and adding features = ["alloc"] to the serde dependency. To preserve the MSRV, I added an std crate feature and enabled it by default, as unfortunately the Error trait hadn't been moved to core at v1.56 yet. So I'm unsure if there are any guidelines if this counts as a breaking change, but I believe not.

I also adjusted the CI to test without default features and with. Let me know if you want me to add that matrix to cargo doc as well.

@daxpedda
Copy link
Author

Apparently serdes v1.69 support for alloc was broken and fixed in serde-rs/serde#1336. So I had to bump the serde dependency to v1.70.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant