Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR updates the project to version 2.3.2 with comprehensive documentation improvements and dependency updates. It modernizes the documentation generation workflow by migrating from DocFX to Kampose, adds namespace documentation files throughout the codebase, and enhances XML documentation with missing <value> tags for properties. The PR also updates test dependencies and refactors the SharedDisposable<T> class to use constructor delegation.
- Migrated from DocFX to Kampose for documentation generation
- Added namespace documentation files across all major namespaces
- Enhanced XML documentation with
<value>tags and improved method documentation - Updated test project dependencies and refined constructor patterns in
SharedDisposable<T>
Reviewed Changes
Copilot reviewed 43 out of 47 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| tests//Kampute..Test.csproj | Updated NuGet test dependencies to newer versions |
| src/Kampute.HttpClient/Utilities/SharedDisposable.cs | Refactored constructor to use delegation pattern and removed nullable factory field |
| src/Kampute.HttpClient/Utilities/ScopedCollection.cs | Added missing typeparam documentation |
| src/**/NamespaceDoc.cs | Added namespace documentation files for API documentation generation |
| src/Kampute.HttpClient/RetryManagement/RetryScheduler.cs | Added <value> tags to property documentation |
| src/Kampute.HttpClient/RetryManagement/BackoffStrategy.cs | Added <value> tag to Strategy property |
| src/Kampute.HttpClient/ErrorHandlers/*.cs | Enhanced documentation with <value> tags and fixed trailing whitespace |
| src/Kampute.HttpClient/Content//.cs | Replaced <inheritdoc/> with explicit method documentation |
| src/Kampute.HttpClient/BackoffStrategies.cs | Added missing <returns> tag and fixed trailing whitespace |
| src/**/Kampute.*.csproj | Bumped version to 2.3.2 and disabled automatic package generation |
| kampose.json | Added new Kampose configuration for documentation generation |
| .github/workflows/main.yml | Migrated CI/CD workflow from DocFX to Kampose with .NET 8.0 |
| README.md | Updated documentation links and contribution guidelines |
| .gitignore | Added .site/ directory and .vscode/ folder exclusions |
| docfx.json | Removed DocFX configuration file (replaced by kampose.json) |
| logo-dark.png, logo-light.png | Added new logo assets for documentation |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 43 out of 47 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This PR updates the project to version 2.3.2 with comprehensive documentation improvements and dependency updates. It modernizes the documentation generation workflow by migrating from DocFX to Kampose, adds namespace documentation files throughout the codebase, and enhances XML documentation with missing tags for properties. The PR also updates test dependencies and refactors the SharedDisposable class to use constructor delegation.