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.

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

cannot use cli (type *"github.com/go-redis/redis".Client) as type *"github.com/nitishm/go-rejson/vendor/github.com/go-redis/redis".Client in argument to rh.SetGoRedisClientgo #30

Closed
tghastings opened this issue Apr 3, 2019 · 7 comments

Comments

@tghastings
Copy link

Error: cannot use cli (type *"github.com/go-redis/redis".Client) as type *"github.com/nitishm/go-rejson/vendor/github.com/go-redis/redis".Client in argument to rh.SetGoRedisClientgo

When trying to run example code on like 97:

rh.SetGoRedisClient(cli)
@ccaum
Copy link

ccaum commented Apr 11, 2019

Same error here.

@nitishm
Copy link
Owner

nitishm commented Apr 16, 2019

@tghastings @ccaum Thanks for reporting this problem. Will address is it as soon as I can.

@tghastings
Copy link
Author

@nitishm, When you fix it could you let me know how? I've been looking at it and they should both be the same type so I'm not sure why they're not.

@nitishm
Copy link
Owner

nitishm commented Apr 19, 2019

@tghastings , @ccaum When you say example code is this referring to the https://github.com/nitishm/go-rejson/blob/master/examples/json_set/json_set.go ?

@HayTran94
Copy link

HayTran94 commented May 22, 2019

Error: cannot use cli (type *"github.com/go-redis/redis".Client) as type *"github.com/nitishm/go-rejson/vendor/github.com/go-redis/redis".Client in argument to rh.SetGoRedisClientgo

When trying to run example code on like 97:

rh.SetGoRedisClient(cli)

You can delete github.com/nitishm/go-rejson/vendor/* folder and enjoy it.

@nicfb
Copy link

nicfb commented Aug 7, 2019

@nitishm - If the solution here was removing the contents of the vendor directory, and that directory was removed when we swapped over to Go Modules, is there anything else that needs to be done here? Or is this a part of a larger underlying problem?

@nitishm
Copy link
Owner

nitishm commented Aug 7, 2019

@nicfb I think the go module switch should solve this problem. Honestly I couldnt reproduce this problem on my own system. But I think it's fair to close this issue with the switch to go modules. I do however want to tag the latest master as v2.1.0, which I will later today.

@nitishm nitishm closed this as completed Aug 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants