Merge pull request #278 from microsoftgraph/feature/trimming #26
Annotations
10 warnings
|
Build and analyze:
src/Microsoft.Graph.Cli.Core/Authentication/ClientCertificateCredentialFactory.cs#L41
Remove this commented out code. (https://rules.sonarsource.com/csharp/RSPEC-125)
|
|
Build and analyze:
src/Microsoft.Graph.Cli.Core/Authentication/ClientCertificateCredentialFactory.cs#L76
Remove this commented out code. (https://rules.sonarsource.com/csharp/RSPEC-125)
|
|
Build and analyze:
src/Microsoft.Graph.Cli.Core/Authentication/ClientCertificateCredentialFactory.cs#L32
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
|
|
Build and analyze:
src/Microsoft.Graph.Cli.Core/Authentication/ClientCertificateCredentialFactory.cs#L40
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
|
|
Build and analyze:
src/Microsoft.Graph.Cli.Core/Authentication/AppOnlyLoginService.cs#L13
'T' is not used in the class. (https://rules.sonarsource.com/csharp/RSPEC-2326)
|
|
Build and analyze:
src/Microsoft.Graph.Cli.Core/Authentication/AuthenticationServiceFactory.cs#L21
Remove this unread private field 'authenticationOptions' or refactor the code to use its value. (https://rules.sonarsource.com/csharp/RSPEC-4487)
|
|
Build and analyze:
src/Microsoft.Graph.Cli.Core/IO/GraphCliClientFactory.cs#L12
Add a 'protected' constructor or the 'static' keyword to the class declaration. (https://rules.sonarsource.com/csharp/RSPEC-1118)
|
|
Build and analyze:
src/Microsoft.Graph.Cli.Core/utils/Constants.cs#L8
Add a 'protected' constructor or the 'static' keyword to the class declaration. (https://rules.sonarsource.com/csharp/RSPEC-1118)
|
|
Build and analyze:
src/Microsoft.Graph.Cli.Core/utils/Constants.cs#L53
Make this field 'private' and encapsulate it in a 'public' property. (https://rules.sonarsource.com/csharp/RSPEC-1104)
|
|
Build and analyze:
src/Microsoft.Graph.Cli.Core/utils/Constants.cs#L53
Change the visibility of 'TenantId' or make it 'const' or 'readonly'. (https://rules.sonarsource.com/csharp/RSPEC-2223)
|
The logs for this run have expired and are no longer available.
Loading