perf: git.exe -> GitReader

[yufeih]

fixes #9088

[codecov]

Codecov Report

Attention: 13 lines in your changes are missing coverage. Please review.

Files	Coverage Δ
src/Docfx.App/RunBuild.cs	54.54% <ø> (-3.79%)	⬇️
src/Docfx.Build.SchemaDriven/OverwriteApplier.cs	89.74% <100.00%> (ø)
src/Docfx.Common/Git/GitRepoInfo.cs	100.00% <ø> (+18.18%)	⬆️
src/Docfx.Common/Git/GitDetail.cs	75.00% <75.00%> (+41.66%)	⬆️
src/Docfx.Dotnet/Parsers/XmlComment.cs	78.82% <0.00%> (ø)
src/Docfx.Common/Git/GitUtility.cs	79.12% <78.00%> (+0.93%)	⬆️

... and 2 files with indirect coverage changes

📢 Thoughts on this report? Let us know!.

[KalleOlaviNiemitalo]

I assume this causes GitReader.dll to be distributed within the docfx .NET tool package. That file seems to be licensed under the Apache-2.0 license; will the distribution satisfy clause 4. (a) of the license terms? AFAICT, the docfx 2.71.1 package does not contain the Apache-2.0 license terms, although some files in it reference the license.

[yufeih]

I assume this causes GitReader.dll to be distributed within the docfx .NET tool package. That file seems to be licensed under the Apache-2.0 license; will the distribution satisfy clause 4. (a) of the license terms? AFAICT, the docfx 2.71.1 package does not contain the Apache-2.0 license terms, although some files in it reference the license.

True. Thanks for the reminder. You are awesome!