-
Notifications
You must be signed in to change notification settings - Fork 61
Conversation
45ae0ed
to
37b0192
Compare
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.
Looks good! Feel free to ignore the comments if these panics are not in scope for this PR.
37b0192
to
1f5289f
Compare
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 like the new conversions much better!
Err(Error::builder(kind).build())? | ||
} | ||
None => Value::Integer(None), | ||
}, |
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.
much better!
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.
Yeah. Not perfect, but at least better...
fn is_json(&self) -> bool; | ||
fn is_enum(&self) -> bool; | ||
fn is_null(&self) -> bool; | ||
} |
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.
💯
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 tried to use these with pg, but it doesn't really fit there that well...
1f5289f
to
af99a3b
Compare
Adresses (at least) these issues: