Skip to content

test: public API verification improvements #163

test: public API verification improvements

test: public API verification improvements #163

Triggered via pull request October 1, 2024 23:06
Status Success
Total duration 6m 20s
Artifacts

codeql-analysis.yml

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

Annotations

12 warnings
Analyze (csharp)
The following actions use a deprecated Node.js version and will be forced to run on node20: github/codeql-action/init@v2, github/codeql-action/analyze@v2. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Analyze (csharp)
CodeQL Action v2 will be deprecated on December 5th, 2024. Please update all occurrences of the CodeQL Action in your workflow files to v3. For more information, see https://github.blog/changelog/2024-01-12-code-scanning-deprecation-of-codeql-action-v2/
Analyze (csharp): src/MockHttp/IInvokedHttpRequestCollection.cs#L8
'IConcurrentReadOnlyCollection<IInvokedHttpRequest>' is obsolete: 'Will be removed in next version.'
Analyze (csharp): src/MockHttp/Internal/Threading/ConcurrentCollection.cs#L6
'IConcurrentReadOnlyCollection<T>' is obsolete: 'Will be removed in next version.'
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/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/IInvokedHttpRequestCollection.cs#L8
'IConcurrentReadOnlyCollection<IInvokedHttpRequest>' is obsolete: 'Will be removed in next version.'
Analyze (csharp): src/MockHttp/Internal/Threading/ConcurrentCollection.cs#L6
'IConcurrentReadOnlyCollection<T>' is obsolete: 'Will be removed in next version.'
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/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/IInvokedHttpRequestCollection.cs#L8
'IConcurrentReadOnlyCollection<IInvokedHttpRequest>' is obsolete: 'Will be removed in next version.'
Analyze (csharp): src/MockHttp/Internal/Threading/ConcurrentCollection.cs#L6
'IConcurrentReadOnlyCollection<T>' is obsolete: 'Will be removed in next version.'