dotnet / docfx

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

Feat/private symbols #9972

Closed Patrick8639 closed 1 month ago

Patrick8639 commented 1 month ago

When specifying "includePrivateMembers" : "true" in docfx.js, the private classes of an assembly are documented, but not the private members of a class.

This fix, modified how the members are enumerated, in order to include the private members of a class, if needed. Unfortunately, the method used to enumerate the members, using Roslyn, doesn't provide a way to enumerate also the private, so I have to set a flag using reflection to do this.

To be correct, code has also been added to not document the class constructor, if any, or compiler-generated members.

codecov[bot] commented 1 month ago

Codecov Report

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

Project coverage is 78.91%. Comparing base (fe673ec) to head (2e86897). Report is 186 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #9972 +/- ## ========================================== + Coverage 74.31% 78.91% +4.59% ========================================== Files 536 540 +4 Lines 23189 23443 +254 Branches 4056 4060 +4 ========================================== + Hits 17234 18501 +1267 + Misses 4853 3804 -1049 - Partials 1102 1138 +36 ```

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