Closed jackterm closed 3 years ago
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot I signed it!
and we'll verify it.
ℹ️ Googlers: Go here for more info.
@googlebot I signed it!
Thank you, this looks good and is a nice addition (it is actually the equivalent of gitHub.com/pborman/uuid.New()
which was the original package (but now is implemented in terms of this package). I think that was one thing people missed.
NewString()
func, which encapsulates the more verboseNew().String()
New().String()
for testing, and it feels ugly. Rather than adding a wrapper aroundgoogle/uuid
, figured it makes sense to add the utility method to the root package