-
Notifications
You must be signed in to change notification settings - Fork 7
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 i128 / u128 #6
base: master
Are you sure you want to change the base?
Conversation
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.
I think I would not want to do 2.x over this.
If there is a more fully featured serde testing library that people use in the meantime, that is all right with me. Serde_test is not special outside of being the crate that serde's test suite has used. But there is not a particular ecosystem value to everybody sticking to the same library as serde's test suite.
Some other things that should be evaluated for a 2.x:
- Order-insensitive comparison, such as being able to test data structures containing HashMap/HashSet, which is not possible today.
- Do
skip_field
calls need to be represented. - Is human_readable integrated in the right way. This is something that was tacked on substantially later than 1.0.
- Examine all the current users of serde_test on crates.io and any lessons learned from that usage.
- Any relevant open issues on the serde repo.
This support is just needed in order to make some comprehensive tests in serde test_suite. Right now I have to avoid testing support for 128-bit integer, which, given that it is supported in serde only partially, makes it much more difficult to identify places where it is absent and what can affect
That's really needed to do at once in one release? It seems to me that it is enough to include that features that's ready, in order of their readiness |
Because since serde-rs/serde#2600 the 128-bit integers are not gated anymore it would be useful to test they support and |
3b647e6
to
d59aef8
Compare
Because MSRV is 1.56 which has support of 128-bit integers, serde_if_integer128 hack is not needed
Because MSRV is 1.56 which has support of 128-bit integers,
serde_if_integer128
hack is not needed.This is breaking change because
Token
enum is not marked asnon_exhaustive
and it is changed.Closes #18