Merge pull request #975 from DFE-Digital/renovate/microsoft-applicati… #8031
DrizzlyOwlcontinuous-integration-dotnet.yml
on: push
build-test
12m 33s
Annotations
11 warnings
|
build-test
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|
|
build-test:
Dfe.ManageFreeSchoolProjects/Dfe.ManageFreeSchoolProjects.UserContext/UserInfo.cs#L41
Possible null reference return.
|
|
build-test:
Dfe.ManageFreeSchoolProjects/Dfe.ManageFreeSchoolProjects.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-test:
Dfe.ManageFreeSchoolProjects/Dfe.ManageFreeSchoolProjects.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-test:
Dfe.ManageFreeSchoolProjects/Dfe.ManageFreeSchoolProjects.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-test:
Dfe.ManageFreeSchoolProjects/Dfe.ManageFreeSchoolProjects.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-test:
Dfe.ManageFreeSchoolProjects/Dfe.ManageFreeSchoolProjects.UserContext/ClientUserInfoService.cs#L24
All 'SetPrincipal' method overloads should be adjacent. (https://rules.sonarsource.com/csharp/RSPEC-4136)
|
|
build-test:
Dfe.ManageFreeSchoolProjects/Dfe.ManageFreeSchoolProjects.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-test:
Dfe.ManageFreeSchoolProjects/Dfe.ManageFreeSchoolProjects.UserContext/ClientUserInfoService.cs#L47
Remove this argument from the method call; it hides the caller information. (https://rules.sonarsource.com/csharp/RSPEC-3236)
|
|
build-test:
Dfe.ManageFreeSchoolProjects/Dfe.ManageFreeSchoolProjects.UserContext/ClientUserInfoService.cs#L65
Remove this unnecessary check for null. (https://rules.sonarsource.com/csharp/RSPEC-2589)
|
|
build-test:
Dfe.ManageFreeSchoolProjects/Dfe.ManageFreeSchoolProjects.UserContext/ClientUserInfoService.cs#L63
Remove this argument from the method call; it hides the caller information. (https://rules.sonarsource.com/csharp/RSPEC-3236)
|