Skip to content

Conversation

@s2quake
Copy link
Owner

@s2quake s2quake commented Nov 2, 2025

Docs: XML IntelliSense + pack-only doc generation

This PR improves developer experience and packaging:

Summary

  • Translate and complete XML documentation for all public APIs in RandomUtility, RandomUtility.Nullable, and RandomUtility.Tuple.
  • Enable XML documentation generation only during dotnet pack to avoid stray XML outputs in dev builds, while ensuring IntelliSense docs ship in the NuGet package.
  • Keep dev builds single-target for faster inner-loop; pack remains multi-target.

Changes

  • docs(random): translate and complete XML docs for RandomUtility, Nullable, and Tuple
  • build(pack): condition pack-only settings at PropertyGroup level
    • Multi-target only during pack
    • Generate XML docs only when packing
    • Include .xml in nupkg output

CI/Release context (existing on main)

  • Code coverage collection via coverlet and upload to Codecov (matrix by TFM)
  • Test reporting via dorny/test-reporter using TRX

Notes

  • No functional code changes; documentation-only plus build configuration for packaging.
  • Validated with dotnet build and dotnet pack; nupkg contains XML docs next to assemblies.

…able, and Tuple

- Add/translate English XML documentation for public APIs
- Ensure <param> and <typeparam> tags are complete and XML is valid
- No functional changes
- Multi-target only during pack
- Generate XML docs only when packing
- Keep dev builds single-target
- Include .xml in nupkg output
@codecov-commenter
Copy link

codecov-commenter commented Nov 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.14%. Comparing base (1b54cc5) to head (25f3f73).
⚠️ Report is 6 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev       #5      +/-   ##
==========================================
+ Coverage   98.93%   99.14%   +0.21%     
==========================================
  Files           4        4              
  Lines         469      469              
  Branches       87       87              
==========================================
+ Hits          464      465       +1     
  Misses          2        2              
+ Partials        3        2       -1     
Flag Coverage Δ
net6.0 99.55% <ø> (ø)
net7.0 99.55% <ø> (ø)
net8.0 99.55% <ø> (ø)
net9.0 99.55% <ø> (ø)
netstandard2.1 98.47% <ø> (-0.22%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

- Fully qualify List/HashSet/SortedSet/Dictionary cref targets to System.Collections.Generic
- Fixes CS0419 ambiguity with method overloads in XML docs
@s2quake s2quake merged commit baf7175 into dev Nov 2, 2025
10 checks passed
@s2quake s2quake deleted the chore/document branch November 2, 2025 04:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants