rename and update mergo module#4378
Conversation
Signed-off-by: Boban Acimovic <boban.acimovic@gmail.com>
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #4378 +/- ##
=======================================
Coverage 59.29% 59.29%
=======================================
Files 288 288
Lines 24769 24769
=======================================
Hits 14687 14687
Misses 9198 9198
Partials 884 884 |
|
Thanks! I'm wondering what motivated the move to a vanity URL; I hope there's good hosting behind it, as I've had bad experience with vanity URLs not always being very reliable |
|
Well, I don't know, I am not the author/maintainer of that module. Since you are using vendoring it is probably not that unsafe. |
There was a problem hiding this comment.
It's not quite inherently safe since we do (among other things) check that vendoring is correct in CI, which requires network access to the module source code. That being said, we should allow GOPROXY for that -- in the past we had it disabled for $REASONS, but it is designed to solve this problem (validation/fetching of code already in your go.sum).
Basically, any issues we have are largely self-inflicted and we can work them out. LGTM!
|
Yeah, it's definitely not a blocker, just bad experience with vanity URLs going AWOL. While |
|
@thaJeztah Allow me to indicate that the hosting is CloudFlare, and I plan to keep the domain. It shouldn't be an issue. |
- What I did
I renamed module
github.com/imdario/mergotodario.cat/mergoand updated to the latest version. This fixes #4377.- How I did it
I renamed the import in
merge.goandmerge_test.goand ranmake vendor.- How to verify it
Run tests.
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)