Skip to content

FAM transfer

FAM transfer #52

Triggered via pull request January 30, 2025 19:48
Status Success
Total duration 2m 31s
Artifacts

dotnet-build.yml

on: pull_request
Build .NET, Test and Analyse
2m 22s
Build .NET, Test and Analyse
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
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#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/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#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.Logging/CorrelationContext.cs#L5
Non-nullable property 'CorrelationId' 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.Logging/LoggerExtensions.cs#L8
Move 'filePath' to the end of the parameter list. (https://rules.sonarsource.com/csharp/RSPEC-3343)
Build .NET, Test and Analyse: src/Frontend/Dfe.Complete.Logging/CorrelationIdMiddleware.cs#L22
Remove this argument from the method call; it hides the caller information. (https://rules.sonarsource.com/csharp/RSPEC-3236)