Skip to content

refactor: replace .NET Core 3.1 with .NET 7 to test .NET Standard 2.1… #234

refactor: replace .NET Core 3.1 with .NET 7 to test .NET Standard 2.1…

refactor: replace .NET Core 3.1 with .NET 7 to test .NET Standard 2.1… #234

Triggered via push October 6, 2024 04:09
Status Success
Total duration 5m 13s
Artifacts

sonarcloud.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
analysis: src/MockHttp/IInvokedHttpRequestCollection.cs#L8
'IConcurrentReadOnlyCollection<IInvokedHttpRequest>' is obsolete: 'Will be removed in next version.'
analysis: src/MockHttp/Internal/Threading/ConcurrentCollection.cs#L6
'IConcurrentReadOnlyCollection<T>' is obsolete: 'Will be removed in next version.'
analysis: src/MockHttp/Internal/HttpCall.cs#L14
Remove this unread private field '_verifiableBecause' or refactor the code to use its value. (https://rules.sonarsource.com/csharp/RSPEC-4487)
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/RequestMatching.cs#L74
Collection-specific "Exists" method should be used instead of the "Any" extension. (https://rules.sonarsource.com/csharp/RSPEC-6605)
analysis: src/MockHttp/Internal/Extensions/ReadOnlyListExtensions.cs#L41
Do not update the loop counter 'i' within the loop body. (https://rules.sonarsource.com/csharp/RSPEC-127)
analysis: src/MockHttp/Internal/Extensions/ReadOnlyListExtensions.cs#L54
Do not update the loop counter 'i' within the loop body. (https://rules.sonarsource.com/csharp/RSPEC-127)
analysis: src/MockHttp/Internal/Threading/AsyncHelpers.cs#L128
Remove this redundant jump. (https://rules.sonarsource.com/csharp/RSPEC-3626)
analysis: src/MockHttp/Matchers/FormDataMatcher.cs#L82
Remove this parameter 'right', whose value is ignored in the method. (https://rules.sonarsource.com/csharp/RSPEC-1172)