Merge pull request #2311 from microsoftgraph/dev #3918
Annotations
10 warnings
|
Build and analyze:
UtilityService/UtilityExtensions.cs#L86
Refactor this method to reduce its Cognitive Complexity from 22 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
|
|
Build and analyze:
UriMatchService/UriTemplateMatcher.cs#L174
Make 'GetQueryExpression' a static method. (https://rules.sonarsource.com/csharp/RSPEC-2325)
|
|
Build and analyze:
UriMatchService/UriTemplateMatcher.cs#L125
Return an empty collection instead of null. (https://rules.sonarsource.com/csharp/RSPEC-1168)
|
|
Build and analyze:
UriMatchService/UriTemplateMatcher.cs#L200
Make 'GetExpression' a static method. (https://rules.sonarsource.com/csharp/RSPEC-2325)
|
|
Build and analyze:
CodeSnippetsReflection/SnippetBaseModel.cs#L34
This line will not be executed conditionally; only the first line of this 3-line block will be. The rest will execute unconditionally. (https://rules.sonarsource.com/csharp/RSPEC-2681)
|
|
Build and analyze:
CodeSnippetsReflection/SnippetBaseModel.cs#L53
"StartsWith" overloads that take a "char" should be used (https://rules.sonarsource.com/csharp/RSPEC-6610)
|
|
Build and analyze:
FileService/Common/FileServiceHelper.cs#L34
Remove this array creation and simply pass the elements. (https://rules.sonarsource.com/csharp/RSPEC-3878)
|
|
Build and analyze:
FileService/Services/AzureBlobStorageUtility.cs#L22
Remove the field '_configuration' and declare it as a local variable in the relevant methods. (https://rules.sonarsource.com/csharp/RSPEC-1450)
|
|
Build and analyze:
FileService/Services/HttpClientUtility.cs#L39
'_httpClient?.SendAsync(requestMessage)' is null on at least one execution path. (https://rules.sonarsource.com/csharp/RSPEC-2259)
|
|
Build and analyze:
FileService/Services/HttpClientUtility.cs#L45
'httpResponseMessage.Content?.ReadAsStringAsync()' is null on at least one execution path. (https://rules.sonarsource.com/csharp/RSPEC-2259)
|
Loading