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.

Sign up for GitHub

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

Jump to bottom

go get error #12

Closed
zanjs opened this issue Feb 14, 2018 · 1 comment
Closed

go get error #12

zanjs opened this issue Feb 14, 2018 · 1 comment

Comments

@zanjs
Copy link

zanjs commented Feb 14, 2018 

PS C:\Users\Anla-E\Desktop> go get github.com/renstrom/shortuuid
# github.com/renstrom/shortuuid
..\go\src\github.com\renstrom\shortuuid\shortuuid.go:21:30: too many arguments in call to DefaultEncoder.Encode
        have (uuid.UUID, error)
        want (uuid.UUID)
..\go\src\github.com\renstrom\shortuuid\shortuuid.go:26:19: too many arguments in call to enc.Encode
        have (uuid.UUID, error)
        want (uuid.UUID)
..\go\src\github.com\renstrom\shortuuid\shortuuid.go:35:17: multiple-value uuid.NewV4() in single-value context
..\go\src\github.com\renstrom\shortuuid\shortuuid.go:49:19: too many arguments in call to enc.Encode
        have (uuid.UUID, error)
        want (uuid.UUID)
@zanjs zanjs closed this as completed Feb 14, 2018
@lithammer
Copy link
Owner

Yeah, unfortunately we're kind of stuck in a bad place until https://github.com/satori/go.uuid either tags a new release, or reverts the backwards incompatible changes to master (see satori/go.uuid#66 and satori/go.uuid#70).

Here are some related issues and pull requests: #7 #8 #9 #10 #11

lithammer pushed a commit that referenced this issue Feb 14, 2018
Use github.com/google/uuid instead of github.com/satori/go.uuid
debc6a4 
Given the recent issues with github.com/satori/go.uuid
(satori/go.uuid#66), which are still partly
unresolved, e.g. its master isn't compatible with us
(see #7, #8, #9, 10, #11 and #12).

Even though github.com/google/uuid is only at v0.1 and clearly states
this:

>This package is currently in development and the API may not be stable.
>
>The API will become stable with v1.

I suspect it's a better longterm investment.
@lithammer lithammer mentioned this issue Feb 14, 2018
Merged
lithammer pushed a commit that referenced this issue Feb 14, 2018
Use github.com/google/uuid instead of github.com/satori/go.uuid
3df12c3 
Given the recent issues with github.com/satori/go.uuid
(satori/go.uuid#66), which are still partly
unresolved, e.g. its master isn't compatible with us
(see #7, #8, #9, 10, #11 and #12).

Even though github.com/google/uuid is only at v0.1 and clearly states
this:

>This package is currently in development and the API may not be stable.
>
>The API will become stable with v1.

I suspect it's a better longterm investment.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lithammer @zanjs