Open
Conversation
* feat: address nullability warnings and improve test methods - Suppress CS1591 warnings in test project for missing XML documentation. - Update response header handling to safely add headers with null checks. - Change test methods from async void to async Task for better async handling. - Update response assertions to use null-forgiving operators and ensure non-null values. - Migrate to required properties in records for DTOs in integration tests. - Replace obsolete HttpRequestMessage.Properties with HttpRequestMessage.Options. - Fix various nullability warnings across integration and utility tests. - Add a comprehensive warnless plan to eliminate all compiler warnings. * feat: enhance nullability handling in FluentHttpHeaders and related classes * feat: update package version to 5.0.0 * chore: remove contributing section from README.md * docs: update README to reflect .NET version changes and add support for net10 * fix: correct typos and improve clarity in README.md * refactor: remove obsolete GqlQuery class and related documentation * chore: remove warnless-plan.md as part of the effort to eliminate compiler and analyzer warnings
* Refactor tests to use Shouldly for assertions - Updated assertions in FluentHttpClientTest to use Shouldly for better readability. - Refactored FluentHttpHeadersTest to replace traditional assertions with Shouldly. - Modified FluentHttpRequestBuilderTest to utilize Shouldly for assertions. - Changed assertions in HttpMiddlewareTest to use Shouldly for consistency. - Updated FileUploadIntegrationTest to use Shouldly for response assertions. - Refactored MessagePackIntegrationTest to utilize Shouldly for assertions. - Changed ResponseCacheIntegrationTest assertions to use Shouldly for clarity. - Updated SerilogIntegrationTest to use Shouldly for null checks and value assertions. - Refactored LoggingHttpMiddlewareTest to use Shouldly for assertions. - Updated TimerHttpMiddlewareTest to utilize Shouldly for assertions. - Added Shouldly package reference to the test project. - Included global using directive for Shouldly in GlobalUsings.cs. * refactor: update test dependencies and improve test assertions
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated release PR for
v5.0.0.Merge this PR to publish the stable release and create the GitHub Release.