rust-embedded-community / serde-json-core

`serde-json` for `no_std` programs
Apache License 2.0
161 stars 59 forks source link

Newtype, variants and unit #42

Closed MathiasKoch closed 3 years ago

MathiasKoch commented 3 years ago

Adds ser/de for the following types:

This is just #25 rebased on master

MathiasKoch commented 3 years ago

Hmm.. Not sure why the CI fails here? @eldruin ?

MathiasKoch commented 3 years ago

Ahh... error[E0658]: non exhaustive is an experimental feature

How should that be handled? Go back to the manual one, and exclude it from clippy checks?

MathiasKoch commented 3 years ago

I think it might be a matter of raising MSRV? non_exhaustive was introduced to stable in 1.40.0

@eldruin would you be okay with me raising MSRV to 1.40.0?

eldruin commented 3 years ago

I would be OK with raising the MSRV to 1.40.0.