-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
[PTRun] add support for UUIDv7 generation #35757
base: main
Are you sure you want to change the base?
Conversation
This comment has been minimized.
This comment has been minimized.
@microsoft-github-policy-service agree |
We will be upgrading to .net 9 shortly. I think they implemented the core logic here to create them. |
Oh, that's great news. Thanks for the heads-up. In that case I'll leave this PR on draft until I can switch out the core implementation with the one from .NET 9 😄 |
#35716 Is the PR for this |
@frederik-hoeft .net 9 PR is now in to the main. |
Summary of the Pull Request
PR Checklist
I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected-> was marked as Help WantedAdded on the required places-> nothing to addDetailed Description of the Pull Request / Additional comments
This PR implements UUIDv7 generation for PowerToys Run as defined in RFC 9562.
Validation Steps Performed
Added unit tests to ensure the version field is correctly set and to assert subsequently generated UUIDs are time-ordered.