Skip to content
.NET

Added API and framework for Conversion CSV export #62

Sign in to view logs

Added API and framework for Conversion CSV export

Added API and framework for Conversion CSV export #62

Triggered via pull request February 3, 2025 14:06
@sukhybhullar-nimblesukhybhullar-nimble
opened #63
feature/187381-csv-export-by-month-view
Status Failure
Total duration 2m 7s
Artifacts –

dotnet-build.yml

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

Annotations

1 error and 10 warnings
Build .NET, Test and Analyse
Process completed with exit code 1.
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/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.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)
Build .NET, Test and Analyse: src/Frontend/Dfe.Complete.UserContext/ClientUserInfoService.cs#L65
Remove this unnecessary check for null. (https://rules.sonarsource.com/csharp/RSPEC-2589)