fix: update to mergo 1.0.0

[caarlos0]

No description provided.

[nibbleshift]

@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: darccio/mergo#245

[andig]

@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

[andig]

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?

[tantinhte]

please merge this PR

[stan-stately]

please merge this. its extremely annoying and its breaking random tooling such as renovate

[phuslu]

I suggest decoupling from mergo to permanently solve this accident. @ALL

[andig]

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

[andig]

@caarlos0 would you mind updating the README, too?

[oliverpool]

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

[caarlos0]

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]

@caarlos0 would you mind updating the README, too?

which part?

[andig]

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.

[oliver-anz]

Bump, this would be very useful thanks

[ripienaar]

A fork is being considered, interested community members should consider adding their efforts there #396