Skip to content

Bump github.com/imdario/mergo reference to v0.3.7 #135

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

Merged
merged 1 commit into from
May 2, 2019

Conversation

unicell
Copy link
Contributor

@unicell unicell commented Apr 9, 2019

For somehow reason, current glide.lock pinned mergo to a very old
version (dates back to 2014), while github.com/Masterminds/sprig
referenced asked for ~v0.3.7 in their glide.yaml which caused build
failure.

This change bumps Mergo to v0.3.7 to satisfy the dependency requirement

Fixes #134

Signed-off-by: Qiu Yu [email protected]

For somehow reason, current glide.lock pinned `mergo` to a very old
version (dates back to 2014), while github.com/Masterminds/sprig
referenced asked for ~v0.3.7 in their glide.yaml which caused build
failure.

This change bumps Mergo to v0.3.7 to satisfy the dependency requirement

Fixes databus23#134

Signed-off-by: Qiu Yu <[email protected]>
@unicell
Copy link
Contributor Author

unicell commented Apr 11, 2019

cc @databus23

@mumoshu
Copy link
Collaborator

mumoshu commented Apr 20, 2019

@unicell Hey! Thanks for your contribution.

I'm willing to merge this, but the only thing I'm unsure is that, why had this not broken our CI?
I'd appreciate it if you could help clarifying it, if possible.
Thanks!

@unicell
Copy link
Contributor Author

unicell commented Apr 23, 2019

@mumoshu I see exactly the opposite. CI is broken [1] already on the tip (c20f260), and the signature matches what #134 describes

[1] https://circleci.com/gh/databus23/helm-diff/25

@mumoshu
Copy link
Collaborator

mumoshu commented May 2, 2019

@unicell Wow, good catch.. Seems like I was blind. Thanks for clarifying, and your support 🙏

@mumoshu mumoshu merged commit f5d745a into databus23:master May 2, 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

Successfully merging this pull request may close these issues.

Fresh clone of the project doesn't build
2 participants