-
Notifications
You must be signed in to change notification settings - Fork 45
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
Context enum #234
Context enum #234
Conversation
Codecov Report
@@ Coverage Diff @@
## main #234 +/- ##
==========================================
+ Coverage 44.64% 46.26% +1.62%
==========================================
Files 70 71 +1
Lines 2249 2475 +226
==========================================
+ Hits 1004 1145 +141
- Misses 1245 1330 +85
Continue to review full report at Codecov.
|
- supports: - Null - String - Number - Unsigned (u64) - Signed (i64) - Float (f64) - List of Values - A lot of the ideas come from https://github.com/dtolnay/serde-yaml/tree/master/src - There is a lot to do till we support more value types and more From implementations for different numbers, but it's good enough for now.
@rawkode anything to add or can I merge? |
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.
The Value type should probably support Map/Object types as well as lists , but I’m happy for that to come as needed.
That's what I intended. |
I'm submitting a
What is the current behaviour?
We use the
Value
struct fromserde_yaml
which is tied to their crate.If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
What is the expected behavior?
We want to have an own implementation of a
Value
class which does not depend on any external crate.What is the motivation / use case for changing the behavior?
Be more independent from outside development
Please tell us about your environment: