ci: switch fully to GitHub actions instead of AppVeyor #2
main.yml
on: pull_request
Annotations
3 errors and 10 warnings
test (ubuntu-latest)
Process completed with exit code 1.
|
test (macos-latest)
Process completed with exit code 1.
|
test (windows-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/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/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/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/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/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)
|
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)
|
Artifacts
Produced during runtime
Name | Size | |
---|---|---|
build
Expired
|
48.1 MB |
|
release-4.1.2-pr83.10
Expired
|
879 KB |
|