Skip to content

feat(net8): add .NET 8 target framework #137

feat(net8): add .NET 8 target framework

feat(net8): add .NET 8 target framework #137

Triggered via pull request November 18, 2023 10:44
@skwasjerskwasjer
opened #86
Status Success
Total duration 4m 34s
Artifacts

sonarcloud.yml

on: pull_request_target
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
analysis: src/MockHttp/Responses/IResponseBehavior.cs#L7
Rename type name ResponseHandlerDelegate so that it does not end in 'Delegate' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1711)
analysis: src/MockHttp/Responses/IResponseBehavior.cs#L22
In virtual/interface member IResponseBehavior.HandleAsync(MockHttpRequestContext, HttpResponseMessage, ResponseHandlerDelegate, CancellationToken), rename parameter next so that it no longer conflicts with the reserved language keyword 'Next'. Using a reserved keyword as the name of a parameter on a virtual/interface member makes it harder for consumers in other languages to override/implement the member. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1716)
analysis: src/MockHttp/Responses/HttpHeaderBehavior.cs#L89
Modify 'TryAdd' to catch a more specific allowed exception type, or rethrow the exception (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1031)
analysis: src/MockHttp/Matchers/HttpHeadersMatcher.cs#L114
'string.Replace(string, string)' has a method overload that takes a 'StringComparison' parameter. Replace this call in 'MockHttp.Matchers.HttpHeadersMatcher.ToString()' with a call to 'string.Replace(string, string, System.StringComparison)' for clarity of intent. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1307)
analysis: src/MockHttp/Http/HttpHeaderEqualityComparer.cs#L70
Call the ArgumentOutOfRangeException constructor that contains a message and/or paramName parameter (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2208)
analysis: src/MockHttp/HttpMockException.cs#L39
'Exception.Exception(SerializationInfo, StreamingContext)' is obsolete: 'This API supports obsolete formatter-based serialization. It should not be called or extended by application code.' (https://aka.ms/dotnet-warnings/SYSLIB0051)
analysis: src/MockHttp/Responses/IResponseBehavior.cs#L7
Rename type name ResponseHandlerDelegate so that it does not end in 'Delegate' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1711)
analysis: src/MockHttp/Responses/IResponseBehavior.cs#L22
In virtual/interface member IResponseBehavior.HandleAsync(MockHttpRequestContext, HttpResponseMessage, ResponseHandlerDelegate, CancellationToken), rename parameter next so that it no longer conflicts with the reserved language keyword 'Next'. Using a reserved keyword as the name of a parameter on a virtual/interface member makes it harder for consumers in other languages to override/implement the member. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1716)
analysis: src/MockHttp/Responses/HttpHeaderBehavior.cs#L89
Modify 'TryAdd' to catch a more specific allowed exception type, or rethrow the exception (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1031)
analysis: src/MockHttp/Http/HttpHeaderEqualityComparer.cs#L70
Call the ArgumentOutOfRangeException constructor that contains a message and/or paramName parameter (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2208)