Skip to content

chore: replace our own TaskHelpers with a more robust impl. #204

chore: replace our own TaskHelpers with a more robust impl.

chore: replace our own TaskHelpers with a more robust impl. #204

Triggered via pull request September 9, 2024 17:15
@skwasjerskwasjer
synchronize #114
Status Success
Total duration 5m 25s
Artifacts

sonarcloud.yml

on: pull_request_target
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
analysis: src/MockHttp/Matchers/RequestUriMatcher.cs#L12
Do not forget to remove this deprecated code someday. (https://rules.sonarsource.com/csharp/RSPEC-1133)
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/Threading/AsyncHelpers.cs#L128
Remove this redundant jump. (https://rules.sonarsource.com/csharp/RSPEC-3626)
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/Matchers/RequestUriMatcher.cs#L12
Do not forget to remove this deprecated code someday. (https://rules.sonarsource.com/csharp/RSPEC-1133)
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/Threading/AsyncHelpers.cs#L128
Remove this redundant jump. (https://rules.sonarsource.com/csharp/RSPEC-3626)
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/Matchers/RequestUriMatcher.cs#L12
Do not forget to remove this deprecated code someday. (https://rules.sonarsource.com/csharp/RSPEC-1133)
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)