Skip to content

Code Quality: Refactored LibGit2 integration as a DI service for future-proofing#18679

Open
Lamparter wants to merge 4 commits into
files-community:mainfrom
Lamparter:libgit2/DI
Open

Code Quality: Refactored LibGit2 integration as a DI service for future-proofing#18679
Lamparter wants to merge 4 commits into
files-community:mainfrom
Lamparter:libgit2/DI

Conversation

@Lamparter

@Lamparter Lamparter commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Resolved / Related Issues

Steps used to test these changes

The implementation is logically identical to the previous, it has just been moved.


In this PR, I:

this future proofs the git abstraction because we will want to move to a DI service later anyway (because the end goal is to allow the user to choose how the app controls git, i.e. they can switch between libgit2sharp and git.exe) and fixes the issue Yair raised before at the same time :)

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.

1 participant