Skip to content
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

replace github.com/gyuho/gocovmerge with github.com/wadey/gocovmerge #16384

Merged
merged 1 commit into from
Aug 7, 2023

Conversation

ahrtr
Copy link
Member

@ahrtr ahrtr commented Aug 7, 2023

Fix #16322

github.com/gyuho/gocovmerge has already been removed or turned into a private repo. I believe it's also forked from github.com/wadey/gocovmerge. So replace it with the original repo github.com/wadey/gocovmerge.

Please read https://github.com/etcd-io/etcd/blob/main/CONTRIBUTING.md#contribution-flow.

@ahrtr ahrtr added dependencies Pull requests that update a dependency file type/cleanup labels Aug 7, 2023
Copy link
Member

@jmhbnz jmhbnz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Note: Found a copy of the repo in waybackmachine: https://web.archive.org/web/20201023134702/https://github.com/gyuho/gocovmerge

Looks like at that point in 2020 it was just two commits ahead of wadey/gocovmerge.

Edit - There are a number of forks of that original repo and this one https://github.com/alexfalkowski/gocovmerge does look to be getting more maintenance so we could consider that option also.

…ovmerge

github.com/gyuho/gocovmerge has already been removed or turned into
a private repo. I believe it's also forked from github.com/wadey/gocovmerge.
So replace it with the original repo github.com/alexfalkowski/gocovmerge.

Signed-off-by: Benjamin Wang <wachao@vmware.com>
@ahrtr
Copy link
Member Author

ahrtr commented Aug 7, 2023

Edit - There are a number of forks of that original repo and this one https://github.com/alexfalkowski/gocovmerge does look to be getting more maintenance so we could consider that option also.

Good point! Updated.

@ahrtr ahrtr merged commit 43f10cb into etcd-io:main Aug 7, 2023
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file type/cleanup
Development

Successfully merging this pull request may close these issues.

"github.com/gyuho/gocovmerge" in tools/mod/go.mod depencies is not a public respository
2 participants