search

dotnet / docfx

Static site generator for .NET API documentation.
https://dotnet.github.io/docfx/
MIT License
3.94k stars 840 forks source link

docs: Fix Docfx.App nuget package usage document #9994

Closed filzrev closed 2 weeks ago

filzrev commented 4 weeks ago

This PR intended to fix issue that reported at #9988.

When using Docfx.App 2.76.0 or later. It need to explicitly add references to Roslyn packages.

Background Roslyn 4.9.2 or later packages contains BuildHost related contentFiles. And these files needs to be copied to output directory.

It works as expected when using ProjectReference. But when using PackageReference and these files are not copied to output directory when packages are referenced as transitive.

To resolve this issue. It need to add direct package references manually. https://github.com/NuGet/Home/issues/6826

codecov[bot] commented 4 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 78.86%. Comparing base (fe673ec) to head (ee060ed). Report is 194 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #9994 +/- ## ========================================== + Coverage 74.31% 78.86% +4.54% ========================================== Files 536 540 +4 Lines 23189 23473 +284 Branches 4056 4066 +10 ========================================== + Hits 17234 18513 +1279 + Misses 4853 3820 -1033 - Partials 1102 1140 +38 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.