chore: bump mergo to v0.3.12

goss-org / goss

Quick and Easy server testing/validation

https://goss.rocks

Apache License 2.0

5.55k stars 473 forks source link

chore: bump mergo to v0.3.12 #740

Closed mbainter closed 2 years ago

mbainter commented 2 years ago

Checklist

[ ] make test-all (UNIX) passes. CI will also test this
[ ] unit and/or integration tests are included (if applicable)
[ ] documentation is changed or added (if applicable)

Description of change

This is a relatively minor update and probably could have been included in #739, but there was enough here that it seemed like it warranted a separate PR. If preferred this PR can be closed and the change added into #739.

ekelali commented 2 years ago

Hello @mbainter , what makes this PR different than #739?

mbainter commented 2 years ago

Hello @mbainter , what makes this PR different than #739?

I'm not particularly familiar with the mergo codebase, and reviewing the changes between 0.3.8 and 0.3.12 seemed a bit riskier, so I was hesitant to include them in a big PR with inconsequential changes where it might get overlooked.

mbainter commented 2 years ago

This should be ready by the way. I don't see any issues with it.

ekelali commented 2 years ago

@mbainter this PR has conflicts

aelsabbahy commented 2 years ago

Hello @mbainter, any update on this PR?

aelsabbahy commented 2 years ago

Closed by #757