Skip to content

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

ci: switch fully to GitHub actions instead of AppVeyor

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

Triggered via pull request November 18, 2023 00:53
Status Failure
Total duration 7m 30s
Artifacts 2

main.yml

on: pull_request
determine-version
11s
determine-version
build (v4.1.2-pr83.10)
2m 0s
build (v4.1.2-pr83.10)
Matrix: test
Fit to window
Zoom out
Zoom in

Annotations

3 errors and 10 warnings
test (ubuntu-latest)
Process completed with exit code 1.
test (windows-latest)
Process completed with exit code 1.
test (macos-latest)
Process completed with exit code 1.
build (v4.1.2-pr83.10): src/MockHttp/Matchers/ContentMatcher.cs#L50
Properties should not return arrays (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1819)
build (v4.1.2-pr83.10): 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)
build (v4.1.2-pr83.10): 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)
build (v4.1.2-pr83.10): src/MockHttp/NetworkLatency.cs#L20
Random is an insecure random number generator. Use cryptographically secure random number generators when randomness is required for security. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca5394)
build (v4.1.2-pr83.10): 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)
build (v4.1.2-pr83.10): 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)
build (v4.1.2-pr83.10): src/MockHttp/MockHttpHandler.cs#L244
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)
build (v4.1.2-pr83.10): 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)
build (v4.1.2-pr83.10): 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)
build (v4.1.2-pr83.10): src/MockHttp/NetworkLatency.cs#L114
Random is an insecure random number generator. Use cryptographically secure random number generators when randomness is required for security. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca5394)

Artifacts

Produced during runtime
Name Size
build Expired
48.2 MB
release-4.1.2-pr83.10 Expired
879 KB