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

Enable usage of different uuid versions #59

Merged
merged 5 commits into from
Feb 14, 2024

Conversation

christian-kolb
Copy link
Contributor

@christian-kolb christian-kolb commented Feb 14, 2024

Changes

  • Dropped final keyword for Id->generateRandom() to enable custom construction of UUID V7 ids.
  • Added documentation for generateRandom override.
  • Adapted docs in preparation for the next release.

Dev setup changes

  • Added caching in CI actions.
  • Added custom docker images to prevent build failures in the CI.

@christian-kolb christian-kolb self-assigned this Feb 14, 2024
Copy link

codecov bot commented Feb 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (d776a0e) 100.00% compared to head (d823ddd) 100.00%.

Additional details and impacted files
@@             Coverage Diff             @@
##                main       #59   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity       118       118           
===========================================
  Files              6         6           
  Lines            245       245           
===========================================
  Hits             245       245           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@christian-kolb christian-kolb mentioned this pull request Feb 14, 2024
@christian-kolb christian-kolb merged commit 17454e8 into main Feb 14, 2024
6 checks passed
@christian-kolb christian-kolb deleted the enable-other-uuid-versions branch February 14, 2024 16:46
@christian-kolb
Copy link
Contributor Author

@scuben Changes are released in 0.12.0 🙂 Thanks for the input.

@plandolt
Copy link
Contributor

Will start using it tomorrow! Thank you and sorry for all the work to remove a handful of characters 🤣

@christian-kolb
Copy link
Contributor Author

No worries, better to have a lot of work with a few characters then putting in hundreds of lines of code that have to be managed indefinitely 😄

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.

2 participants