-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Update JMESPath dependency #2284
Conversation
Hi @thepudds, this PR updates the SDK's |
41241be
to
d643294
Compare
d643294
to
72af164
Compare
Updates Go dep's lock file to contain the correct tracking hash. The Gopkg.toml file was updated in #2284 but the lock file was missed.
I'm still having problems with this (using modules):
and the latest one seems to be loaded by default by both v1 and v2 of the sdk. |
Thanks for the update @shaunc I'm having difficulties reproducing this issue. Could you try cleaning your Go cache, and modcache? With a clean cache and getting the latest, or specific commit go-jmespath i'm unable to reproduce any error. go clean -cache
go clean -modcache
go get github.com/jmespath/go-jmespath@latest |
@jasdel -- In fact, that worked! Sorry -- I'm a go newbie... if this were npm I would have thought of the analogous on my own. :) Thank you! |
Great, glad that resolved the issue! |
For others coming from search engines. I was able to fix
by running
Which probably was overkill, but worked 🤷♂ |
Updates
github.com/jmespath/go-jmespath
to the latest version of JMESPath.Fix #1907 #2266