Skip to content

Force failed checkouts while fetching refs#5296

Merged
adamzip merged 1 commit intodotnet:mainfrom
adamzip:fix-failed-checkout
Sep 23, 2025
Merged

Force failed checkouts while fetching refs#5296
adamzip merged 1 commit intodotnet:mainfrom
adamzip:fix-failed-checkout

Conversation

@adamzip
Copy link
Copy Markdown
Contributor

@adamzip adamzip commented Sep 23, 2025

Copilot AI review requested due to automatic review settings September 23, 2025 10:56
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds force checkout functionality to the git operations by introducing a force parameter to checkout methods and enabling forced checkouts during repository preparation in the Virtual Mono Repo CloneManager.

  • Adds optional force parameter to all checkout-related methods across the git client interfaces and implementations
  • Updates CloneManager to use forced checkout after cleaning the repository to handle scenarios where checkout might fail due to conflicting local changes

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/Microsoft.DotNet.Darc/DarcLib/VirtualMonoRepo/CloneManager.cs Updates checkout call to use force parameter after repository cleaning
src/Microsoft.DotNet.Darc/DarcLib/LocalGitRepo.cs Adds force parameter to CheckoutAsync method signature and implementation
src/Microsoft.DotNet.Darc/DarcLib/LocalGitClient.cs Implements force checkout by adding -f flag to git checkout command
src/Microsoft.DotNet.Darc/DarcLib/ILocalGitRepo.cs Updates interface to include force parameter in CheckoutAsync method
src/Microsoft.DotNet.Darc/DarcLib/ILocalGitClient.cs Updates interface to include force parameter in CheckoutAsync method

Comment thread src/Microsoft.DotNet.Darc/DarcLib/LocalGitClient.cs Outdated
@adamzip adamzip force-pushed the fix-failed-checkout branch from ea98671 to d5144bc Compare September 23, 2025 11:20
@adamzip adamzip merged commit 9e0f885 into dotnet:main Sep 23, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants