Fix some timestamp generation #691

Merged

KodrAus merged 6 commits from fix/timestamp-gen into main 2023-06-26 20:32:11 -05:00

Conversation 0 Commits 6 Files changed 12 +566 -77

KodrAus commented 2023-06-26 20:15:06 -05:00 (Migrated from github.com)

• Copy link
• Copy Markdown

Closes #680

This PR makes Timestamp conversions explicitly wrap rather than panic on overflow.

I've also fixed up a warning in arbitrary, and removed a random dbg! statement that shouldn't be there.

Closes #680 This PR makes `Timestamp` conversions explicitly wrap rather than panic on overflow. I've also fixed up a warning in `arbitrary`, and removed a random `dbg!` statement that shouldn't be there.

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

breaking

This causes a breaking change and will be merged to breaking branch.

  

bug

  

dependencies

Pull requests that update a dependency file

  

good first issue

  

help wanted

  

unstable

Tracking issue for an unstable optional dependency

No labels

breaking

bug

dependencies

good first issue

help wanted

unstable

Milestone

Clear milestone

No items

No milestone

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

gh-uuid-rs/uuid!691

No description provided.