Skip to content

ci: switch fully to GitHub actions instead of AppVeyor #38

ci: switch fully to GitHub actions instead of AppVeyor

ci: switch fully to GitHub actions instead of AppVeyor #38

Triggered via pull request November 18, 2023 00:17
Status Failure
Total duration 4m 27s
Artifacts

codeql-analysis.yml

on: pull_request
Matrix: Analyze
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 10 warnings
Analyze (csharp): src/MockHttp/Extensions/RequestMatchingExtensions.cs#L200
The type or namespace name 'HttpMethod' could not be found (are you missing a using directive or an assembly reference?)
Analyze (csharp): src/MockHttp/Extensions/RequestMatchingExtensions.cs#L699
The type or namespace name 'HttpRequestMessage' could not be found (are you missing a using directive or an assembly reference?)
Analyze (csharp): src/MockHttp/Extensions/HttpContentExtensions.cs#L13
The type or namespace name 'ByteArrayContent' could not be found (are you missing a using directive or an assembly reference?)
Analyze (csharp): src/MockHttp/IInvokedHttpRequest.cs#L13
The type or namespace name 'HttpRequestMessage' could not be found (are you missing a using directive or an assembly reference?)
Analyze (csharp): src/MockHttp/Extensions/HttpContentExtensions.cs#L14
The type or namespace name 'HttpContent' could not be found (are you missing a using directive or an assembly reference?)
Analyze (csharp): src/MockHttp/Extensions/IRespondsExtensions.cs#L19
The type or namespace name 'HttpResponseMessage' could not be found (are you missing a using directive or an assembly reference?)
Analyze (csharp): src/MockHttp/Extensions/IRespondsExtensions.cs#L35
The type or namespace name 'HttpRequestMessage' could not be found (are you missing a using directive or an assembly reference?)
Analyze (csharp): src/MockHttp/Extensions/IRespondsExtensions.cs#L35
The type or namespace name 'HttpResponseMessage' could not be found (are you missing a using directive or an assembly reference?)
Analyze (csharp): src/MockHttp/Extensions/IRespondsExtensions.cs#L132
The type or namespace name 'HttpResponseMessage' could not be found (are you missing a using directive or an assembly reference?)
Analyze (csharp): src/MockHttp/InvokedHttpRequest.cs#L17
The type or namespace name 'HttpRequestMessage' could not be found (are you missing a using directive or an assembly reference?)
Analyze (csharp): src/MockHttp/Matchers/ContentMatcher.cs#L50
Properties should not return arrays (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1819)
Analyze (csharp): 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)
Analyze (csharp): src/MockHttp/Threading/ConcurrentCollection.cs#L79
Exception type System.IndexOutOfRangeException is reserved by the runtime (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2201)
Analyze (csharp): 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)
Analyze (csharp): src/MockHttp/Extensions/ResponseBuilderExtensions.cs#L204
Consider calling ConfigureAwait on the awaited task (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2007)
Analyze (csharp): 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)
Analyze (csharp): src/MockHttp/Responses/HttpHeaderBehavior.cs#L10
Change type of field 'HeadersWithSingleValueOnly' from 'System.Collections.Generic.ISet<string>' to 'System.Collections.Generic.HashSet<string>' for improved performance (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1859)
Analyze (csharp): src/MockHttp/Http/HttpHeadersCollection.cs#L46
Prefer 'static readonly' fields over constant array arguments if the called method is called repeatedly and is not mutating the passed array (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1861)
Analyze (csharp): src/MockHttp/Http/HttpHeadersCollection.cs#L64
Prefer 'static readonly' fields over constant array arguments if the called method is called repeatedly and is not mutating the passed array (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1861)
Analyze (csharp): src/MockHttp/RequestMatching.cs#L73
Prefer comparing 'Count' to 0 rather than using 'Any()', both for clarity and for performance (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1860)