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

gitindex: update remote.origin.url if clone already exists #776

Merged
merged 1 commit into from
May 3, 2024

Conversation

keegancsmith
Copy link
Member

This is so that when authentication details change we correctly update the CloneURL for repositories managed by our mirror commands.

Alternative to #775

This is so that when authentication details change we correctly update
the CloneURL for repositories managed by our mirror commands.
@xavier-calland
Copy link
Contributor

I think your version is better. This does not duplicate calling the git command.
Go is not a language I work on regularly, so I lack knowledge of the possibilities and best practices.

@keegancsmith keegancsmith merged commit 7c5b778 into main May 3, 2024
9 checks passed
@keegancsmith keegancsmith deleted the k/clone-update branch May 3, 2024 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants