Update dependency DocumentFormat.OpenXml to 3.1.1 #7978
Triggered via pull request
November 15, 2024 17:38
Status
Success
Total duration
11m 26s
Artifacts
–
continuous-integration-dotnet.yml
on: pull_request
build-test
11m 16s
Annotations
10 warnings
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/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.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/UserInfo.cs#L32
"Array.Find" static method should be used instead of the "FirstOrDefault" extension method. (https://rules.sonarsource.com/csharp/RSPEC-6602)
|
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)
|