Skip to content

fix: retry with git checkout#2147

Merged
ludamad merged 1 commit into
masterfrom
ad/retry-git-checkout
Sep 8, 2023
Merged

fix: retry with git checkout#2147
ludamad merged 1 commit into
masterfrom
ad/retry-git-checkout

Conversation

@ludamad

@ludamad ludamad commented Sep 8, 2023

Copy link
Copy Markdown
Collaborator

Works around rate limiting on git. We may want to up this count but it is not a bad idea to still have retries. Turns out this hits the network. Not that surprising in retrospect (git checkout can sync to find branches on the remote), but I usually think of it in the fully local DB case so didn't think of adding this before

Works around rate limiting on git. We may want to up this count but it is not a bad idea to still have retries. Turns out this hits the network. Not that surprising in retrospect (git checkout can sync to find branches on the remote), but I usually think of it in the fully local DB case so didn't think of adding this before
@ludamad ludamad requested a review from charlielye September 8, 2023 18:30
@ludamad ludamad merged commit 9df0431 into master Sep 8, 2023
@ludamad ludamad deleted the ad/retry-git-checkout branch September 8, 2023 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants