-
-
Notifications
You must be signed in to change notification settings - Fork 22
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
uuid v9 #12
Comments
Yes, please PR! |
PR raised here: #13. Weirdly don't appear able to add reviewers to the PR. |
merged!
Debdut Karmakar
calendly.com/debdut
@KarmakarDebdut <https://twitter.com/KarmakarDebdut>
Github <https://github.com/debdut>
ProductHunt ***@***.***>
…On Oct 21, 2022 at 3:04:56 PM, Katerine Knox ***@***.***> wrote:
PR raised here: #13 <#13>.
Weirdly don't appear able to add reviewers to the PR.
—
Reply to this email directly, view it on GitHub
<#12 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABZV63RXNOPQG2MBAZFQFZ3WEJPUBANCNFSM6AAAAAARH5VUIE>
.
You are receiving this because you modified the open/close state.Message
ID: ***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hello,
Thanks for the library. I was wondering if there was any appetite for updating the
uuid
version to v9? Or havinguuid
specified as a peer dependency that would support v8 and v9.UUID Changelog here: https://github.com/uuidjs/uuid/blob/main/CHANGELOG.md
The breaking changes are mainly around dropping support for legacy browsers and node versions so I don't believe upgrading would require any code changes. Would be happy to raise a PR if there are no objections to upgrading?
The text was updated successfully, but these errors were encountered: