Skip to content
This repository was archived by the owner on Jun 5, 2025. It is now read-only.

GitExecute changes for WSL - #3839

Merged
ssparach merged 6 commits into
mainfrom
user/ssparach/WSLIntegration
Sep 19, 2024
Merged

GitExecute changes for WSL#3839
ssparach merged 6 commits into
mainfrom
user/ssparach/WSLIntegration

Conversation

@ssparach

@ssparach ssparach commented Sep 11, 2024

Copy link
Copy Markdown
Contributor

Summary of the pull request

This PR modifies the arguments used while starting a process inside GitExecute to accommodate obtaining git information for WSL repositories.

References and relevant issues

Detailed description of the pull request / Additional comments

This PR adds:

  • WSLIntegrator model class: parses WSL paths to extract relevant information and arguments
  • WSLIntegratorUnitTests: unit tests for WSLIntegrator

This PR modifies GitExecute to invoke wsl.exe with the expected arguments to obtain git information for registered wsl repository paths.

Validation steps performed

Package build
Unit Tests
Manual testing including registration of repositories inside Dev Home for the git extension

PR checklist

  • Closes #xxx
  • Tests added/passed
  • Documentation updated

Comment thread extensions/GitExtension/FileExplorerGitIntegration/Models/WslIntegrator.cs Outdated
Comment thread extensions/GitExtension/FileExplorerGitIntegration/Models/WslIntegrator.cs Outdated
Comment thread extensions/GitExtension/FileExplorerGitIntegration/Models/GitExecute.cs Outdated
Comment thread extensions/GitExtension/FileExplorerGitIntegration/Models/GitExecute.cs Outdated
Comment thread extensions/GitExtension/FileExplorerGitIntegration/Models/GitExecute.cs Outdated
@ssparach
ssparach requested a review from bbonaby September 19, 2024 21:30
@ssparach
ssparach merged commit c60dc85 into main Sep 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

File Explorer version control integration should work with WSL repos/paths

4 participants