FAM transfer #51
dotnet-build.yml
on: pull_request
Build .NET, Test and Analyse
2m 27s
Annotations
10 warnings
Build .NET, Test and Analyse:
src/Core/Dfe.Complete.Utils/EnumExtensions.cs#L39
Converting null literal or possible null value to non-nullable type.
|
Build .NET, Test and Analyse:
src/Frontend/Dfe.Complete.UserContext/ClientUserInfoService.cs#L14
Non-nullable property 'UserInfo' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
Build .NET, Test and Analyse:
src/Frontend/Dfe.Complete.UserContext/UserInfo.cs#L41
Possible null reference return.
|
Build .NET, Test and Analyse:
src/Frontend/Dfe.Complete.UserContext/UserInfo.cs#L8
Non-nullable property 'Name' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
Build .NET, Test and Analyse:
src/Frontend/Dfe.Complete.UserContext/UserInfo.cs#L9
Non-nullable property 'Roles' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
Build .NET, Test and Analyse:
src/Frontend/Dfe.Complete.UserContext/ClientUserInfoService.cs#L24
All 'SetPrincipal' method overloads should be adjacent. (https://rules.sonarsource.com/csharp/RSPEC-4136)
|
Build .NET, Test and Analyse:
src/Frontend/Dfe.Complete.UserContext/ClientUserInfoService.cs#L47
Remove this argument from the method call; it hides the caller information. (https://rules.sonarsource.com/csharp/RSPEC-3236)
|
Build .NET, Test and Analyse:
src/Frontend/Dfe.Complete.UserContext/ServerUserInfoService.cs#L21
Indexing at 0 should be used instead of the "Enumerable" extension method "First" (https://rules.sonarsource.com/csharp/RSPEC-6608)
|
Build .NET, Test and Analyse:
src/Frontend/Dfe.Complete.UserContext/ClientUserInfoService.cs#L63
Remove this argument from the method call; it hides the caller information. (https://rules.sonarsource.com/csharp/RSPEC-3236)
|
Build .NET, Test and Analyse:
src/Frontend/Dfe.Complete.UserContext/ClientUserInfoService.cs#L61
Make 'GetPrincipalName' a static method. (https://rules.sonarsource.com/csharp/RSPEC-2325)
|