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

Adding Timestamp decode/encode #351

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

Conversation

junderw
Copy link

@junderw junderw commented Aug 18, 2024

This is a baseline just to have something to work off of, very WIP.

  1. Should we perhaps just use DateTime from the chrono crate under a feature flag instead of creating a simple enum. (I've decided against this, it's too much of a hassle.)
  2. How should we go about including it in rmp-serde if at all?
  3. Checking for invalid values (ie. nanos above 999_999_999) should probably be done better. I am not too familiar with this library and how I should be propagating such errors.
  4. Also, please advise on any formatting requests. (I turned off rustfmt)

Let me know if this isn't something you're interested it.

@junderw junderw marked this pull request as draft August 18, 2024 15:00
rmp/src/decode/mod.rs Outdated Show resolved Hide resolved
@junderw junderw changed the title WIP: Adding Timestamp decode/encode Adding Timestamp decode/encode Aug 26, 2024
@junderw junderw marked this pull request as ready for review August 26, 2024 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant