-
-
Notifications
You must be signed in to change notification settings - Fork 132
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] Added the polyfill #197
Conversation
47e8c87
to
e3cf6b8
Compare
92e68e2
to
2734db4
Compare
I think this PR is ready 🎉 |
based on #197 (comment) (and my answer just after), it is not, as the error handling is not the same in the polyfill and in the original extension. |
84c241b
to
c12cfb3
Compare
OK, I removed all the type hints and codes incompatible with PHP > 5.3 Should be really ready now 😄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a minor comment regarding the node part of time-based UUIDs, to avoid possible conflicts with MAC-based node values.
Thank you @lyrixx. |
This PR was squashed before being merged into the 1.12-dev branch. Discussion ---------- [Uuid] Added the polyfill Commits ------- 9b6d18b [Uuid] Added the polyfill
No description provided.