Closed caarlos0 closed 1 month ago
@mattfarina Can we get this merged? This fixes a larger issue rather than just bumping up the version of mergo 1.0.0. Mergo switched their repo url from a gh url to a vanity url. See here: https://github.com/darccio/mergo/issues/245
@mattfarina please merge. The current experience is rather painful, even when that's an upstream issue:
go get -u
go: github.com/imdario/mergo@v1.0.0: parsing go.mod:
module declares its path as: dario.cat/mergo
but was required as: github.com/imdario/mergo
trying github.com/imdario/mergo@v0.3.16
Note that readme also has this:
Sprig leverages mergo to handle merges. In its v0.3.9 release, there was a behavior change that impacts merging template functions in sprig. It is currently recommended to use v0.3.10 or later of that package. Using v0.3.9 will cause sprig tests to fail.
Not sure if upgrading to post 0.3.11 would break anything? Is fixing this a prerequisite?
please merge this PR
please merge this. its extremely annoying and its breaking random tooling such as renovate
I suggest decoupling from mergo to permanently solve this accident. @all
Using v0.3.9 will cause sprig tests to fail.
@mattfarina I've re-run the tests on mergo 1.0- all fine:
sprig mergo
❯ grb master
Current branch mergo is up to date.
sprig mergo
❯ got ./...
ok github.com/Masterminds/sprig/v3 4.329s
@caarlos0 would you mind updating the README, too?
For anyone impacted, add the following line at the bottom of go.mod
in the meantime:
replace github.com/imdario/mergo => dario.cat/mergo v1.0.0
For anyone impacted, add the following line at the bottom of
go.mod
in the meantime:replace github.com/imdario/mergo => dario.cat/mergo v1.0.0
FWIW this doesn't actually work if your users do go install your-tool
@caarlos0 would you mind updating the README, too?
which part?
I think this needs be removed if no longer an issue:
IMPORTANT NOTES Sprig leverages mergo to handle merges. In its v0.3.9 release, there was a behavior change that impacts merging template functions in sprig. It is currently recommended to use v0.3.10 or later of that package. Using v0.3.9 will cause sprig tests to fail.
Bump, this would be very useful thanks
A fork is being considered, interested community members should consider adding their efforts there https://github.com/Masterminds/sprig/issues/396
Thanks for pointing this out. Since I'm planning on a release, I created #406 to update to the 1.0.1 release of mergo and to handle the conflicts.
Thanks for pointing this out. Since I'm planning on a release, I created #406 to update to the 1.0.1 release of mergo and to handle the conflicts.
excellent news that a release is coming, would you be in a position to make some changes like remove the deprecated crypto algo functions by any chance?
Would really like to see sprig continue since the available forks have not really panned out - and forks aren't in anyones interest
@ripienaar crypto functions can be deprecated but not removed without making a major API change. Consumers of this library enable those crypto functions to be used so I'm hesitant to remove them. This is why adding something new is always something we are hesitant of. You need to keep those APIs around for a long time.
Imagine if tools like Helm had template functions just vanish.
Closing this PR as the intended change has already landed.
The ones I am talking about @mattfarina are the ones Go - and industry at large - have stated are unsafe, shouldn't be used etc