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.

Sign up for GitHub

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

Jump to bottom

Support for UUIDv7 #120

Merged
merged 7 commits into from
Jun 12, 2024
Merged

Support for UUIDv7 #120

merged 7 commits into from
Jun 12, 2024

Conversation

nghduc97
Copy link
Contributor

@nghduc97 nghduc97 commented Jun 7, 2024 •
edited
Loading

TODOs:

  • UUIDv7 generation.
  • Extract timestamp from UUIDv7.
  • Add tests.
  • Update docs.

Related issue: #116

@nghduc97 nghduc97 changed the title Support for UUIDv7 (WIP) Support for UUIDv7 Jun 7, 2024
@nalgeon
Copy link
Owner

nalgeon commented Jun 8, 2024

Thank you! I'm a bit busy right now, will probably review the PR in the next few weeks.

@nghduc97
Copy link
Contributor Author

nghduc97 commented Jun 9, 2024

Okay, sure

@LifeCANvs LifeCANvs mentioned this pull request Jun 10, 2024
Open
@nalgeon nalgeon merged commit 1ba95e4 into nalgeon:main Jun 12, 2024
3 checks passed
@nalgeon
Copy link
Owner

nalgeon commented Jun 12, 2024

Thanks a lot! This is such a great PR: code, tests, documentation — everything is well done and in place 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nghduc97 @nalgeon