fix: address some static code analysis issues, exposed by the latest analyzer #13
Annotations
10 warnings
|
Run dotnet build --no-restore -c Release /p:Version="4.2.1-pr87.8" /p:PackageVersion="4.2.1-pr87.8" /p:AssemblyVersion="4.2.1.0" /p:FileVersion="4.2.1.0" /p:InformationalVersion="4.2.1-pr87.8+Branch.pull-87-merge.Sha.5cc69cba95ebb3dc861a847dd507e06ff8d4f593":
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)
|
|
Run dotnet build --no-restore -c Release /p:Version="4.2.1-pr87.8" /p:PackageVersion="4.2.1-pr87.8" /p:AssemblyVersion="4.2.1.0" /p:FileVersion="4.2.1.0" /p:InformationalVersion="4.2.1-pr87.8+Branch.pull-87-merge.Sha.5cc69cba95ebb3dc861a847dd507e06ff8d4f593":
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)
|
|
Run dotnet build --no-restore -c Release /p:Version="4.2.1-pr87.8" /p:PackageVersion="4.2.1-pr87.8" /p:AssemblyVersion="4.2.1.0" /p:FileVersion="4.2.1.0" /p:InformationalVersion="4.2.1-pr87.8+Branch.pull-87-merge.Sha.5cc69cba95ebb3dc861a847dd507e06ff8d4f593":
src/MockHttp/Http/DataEscapingHelper.cs#L53
'string.Replace(string, string?)' has a method overload that takes a 'StringComparison' parameter. Replace this call in 'MockHttp.Http.DataEscapingHelper.UnescapeData(string)' with a call to 'string.Replace(string, string?, System.StringComparison)' for clarity of intent. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1307)
|
|
Run dotnet build --no-restore -c Release /p:Version="4.2.1-pr87.8" /p:PackageVersion="4.2.1-pr87.8" /p:AssemblyVersion="4.2.1.0" /p:FileVersion="4.2.1.0" /p:InformationalVersion="4.2.1-pr87.8+Branch.pull-87-merge.Sha.5cc69cba95ebb3dc861a847dd507e06ff8d4f593":
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)
|
|
Run dotnet build --no-restore -c Release /p:Version="4.2.1-pr87.8" /p:PackageVersion="4.2.1-pr87.8" /p:AssemblyVersion="4.2.1.0" /p:FileVersion="4.2.1.0" /p:InformationalVersion="4.2.1-pr87.8+Branch.pull-87-merge.Sha.5cc69cba95ebb3dc861a847dd507e06ff8d4f593":
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)
|
|
Run dotnet build --no-restore -c Release /p:Version="4.2.1-pr87.8" /p:PackageVersion="4.2.1-pr87.8" /p:AssemblyVersion="4.2.1.0" /p:FileVersion="4.2.1.0" /p:InformationalVersion="4.2.1-pr87.8+Branch.pull-87-merge.Sha.5cc69cba95ebb3dc861a847dd507e06ff8d4f593":
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)
|
|
Run dotnet build --no-restore -c Release /p:Version="4.2.1-pr87.8" /p:PackageVersion="4.2.1-pr87.8" /p:AssemblyVersion="4.2.1.0" /p:FileVersion="4.2.1.0" /p:InformationalVersion="4.2.1-pr87.8+Branch.pull-87-merge.Sha.5cc69cba95ebb3dc861a847dd507e06ff8d4f593":
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)
|
|
Run dotnet build --no-restore -c Release /p:Version="4.2.1-pr87.8" /p:PackageVersion="4.2.1-pr87.8" /p:AssemblyVersion="4.2.1.0" /p:FileVersion="4.2.1.0" /p:InformationalVersion="4.2.1-pr87.8+Branch.pull-87-merge.Sha.5cc69cba95ebb3dc861a847dd507e06ff8d4f593":
src/MockHttp/HttpMockException.cs#L39
Provide a message for the ObsoleteAttribute that marks .ctor as Obsolete (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1041)
|
|
Run dotnet build --no-restore -c Release /p:Version="4.2.1-pr87.8" /p:PackageVersion="4.2.1-pr87.8" /p:AssemblyVersion="4.2.1.0" /p:FileVersion="4.2.1.0" /p:InformationalVersion="4.2.1-pr87.8+Branch.pull-87-merge.Sha.5cc69cba95ebb3dc861a847dd507e06ff8d4f593":
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)
|
|
Run dotnet build --no-restore -c Release /p:Version="4.2.1-pr87.8" /p:PackageVersion="4.2.1-pr87.8" /p:AssemblyVersion="4.2.1.0" /p:FileVersion="4.2.1.0" /p:InformationalVersion="4.2.1-pr87.8+Branch.pull-87-merge.Sha.5cc69cba95ebb3dc861a847dd507e06ff8d4f593":
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)
|
The logs for this run have expired and are no longer available.
Loading