Skip to content

Fix #57910 - Add setting for auto fetch timeout#59988

Merged
joaomoreno merged 1 commit intomicrosoft:masterfrom
w9jds:git-autofetch-period
Jan 4, 2019
Merged

Fix #57910 - Add setting for auto fetch timeout#59988
joaomoreno merged 1 commit intomicrosoft:masterfrom
w9jds:git-autofetch-period

Conversation

@w9jds
Copy link
Contributor

@w9jds w9jds commented Oct 4, 2018

Finishes the race condition when either setting is changed, and then if it wasn’t disabled creates a new timeout using the new value (instead of waiting for it to go through the loop again to update).

Finishes the race condition when either setting is changed, and then if it wasn’t disabled creates a new timeout using the new value (instead of waiting for it to go through the loop again to update).
@msftclas
Copy link

msftclas commented Oct 4, 2018

CLA assistant check
All CLA requirements met.

@joaomoreno joaomoreno added the git GIT issues label Oct 9, 2018
@joaomoreno joaomoreno added this to the Backlog milestone Oct 9, 2018
@joaomoreno
Copy link
Member

Changed the unit to seconds instead of minutes. Also cleaned up all that complexity of resetting the machine when that value changes. It's more than OK to just wait until the next auto fetch gets triggered to trigger a new one with the new delay.

Thanks! 🎆

@joaomoreno joaomoreno merged commit 7761add into microsoft:master Jan 4, 2019
@w9jds w9jds deleted the git-autofetch-period branch October 6, 2019 05:23
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

git GIT issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants