Refactor user group name handling in TfsUserMappingTool #1059
MrHinshmain.yml
on: pull_request
Setup & Configuration
1m 23s
Build Run Data
7s
Release to Docs
44s
Create Elmah.io Deployment
0s
Release to Marketplace
0s
Release to Chocolatey
0s
Release to Winget
0s
Annotations
11 warnings
|
Build, Test, Sonar Cloud Analysis, & Package
No files were found with the provided path: ./docs/Reference/Generated/**. No artifacts will be uploaded.
|
|
Build, Test, Sonar Cloud Analysis, & Package:
src/MigrationTools.Clients.TfsObjectModel/Tools/TfsUserMappingTool.cs#L100
Don't use string interpolation in logging message templates. (https://rules.sonarsource.com/csharp/RSPEC-2629)
|
|
Build, Test, Sonar Cloud Analysis, & Package:
src/MigrationTools.Clients.TfsObjectModel/Tools/TfsUserMappingTool.cs#L65
Don't use string interpolation in logging message templates. (https://rules.sonarsource.com/csharp/RSPEC-2629)
|
|
Build, Test, Sonar Cloud Analysis, & Package:
src/MigrationTools.Clients.TfsObjectModel/Tools/TfsUserMappingTool.cs#L77
Loops should be simplified using the "Where" LINQ method (https://rules.sonarsource.com/csharp/RSPEC-3267)
|
|
Build, Test, Sonar Cloud Analysis, & Package:
src/MigrationTools.Clients.TfsObjectModel/Tools/TfsUserMappingTool.cs#L81
Merge this if statement with the enclosing one. (https://rules.sonarsource.com/csharp/RSPEC-1066)
|
|
Build, Test, Sonar Cloud Analysis, & Package:
src/MigrationTools.Clients.TfsObjectModel/Tools/TfsUserMappingTool.cs#L65
Logging in a catch clause should pass the caught exception as a parameter. (https://rules.sonarsource.com/csharp/RSPEC-6667)
|
|
Build, Test, Sonar Cloud Analysis, & Package:
src/MigrationTools.Clients.TfsObjectModel/Tools/TfsUserMappingTool.cs#L70
Refactor this method to reduce its Cognitive Complexity from 16 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
|
|
Build, Test, Sonar Cloud Analysis, & Package:
src/MigrationTools.Clients.TfsObjectModel/Tools/TfsUserMappingTool.cs#L46
Use PascalCase for named placeholders. (https://rules.sonarsource.com/csharp/RSPEC-6678)
|
|
Build, Test, Sonar Cloud Analysis, & Package:
src/MigrationTools.Clients.TfsObjectModel/Tools/TfsUserMappingTool.cs#L22
Remove this unnecessary cast to 'TfsUserMappingToolOptions'. (https://rules.sonarsource.com/csharp/RSPEC-1905)
|
|
Build, Test, Sonar Cloud Analysis, & Package:
src/MigrationTools.Clients.TfsObjectModel/Processors/TfsWorkItemMigrationProcessor.cs#L429
The variable 'ex' is declared but never used
|
|
Build, Test, Sonar Cloud Analysis, & Package
ProjectReference 'D:\a\azure-devops-migration-tools\azure-devops-migration-tools\src\MigrationTools.Clients.TfsObjectModel\MigrationTools.Clients.TfsObjectModel.csproj' was resolved using '.NETFramework,Version=v4.7.2' instead of the project target framework '.NETCoreApp,Version=v8.0'. This project may not be fully compatible with your project.
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
AzureDevOpsMigrationTools-Packages
|
23.9 MB |
sha256:320d824b21f6592ce6412509d054f41d61a60ba1b99cd0da1bbe5ebb0a89eded
|
|
|
AzureDevOpsMigrationTools-Scripts
|
25.7 KB |
sha256:2a97e57cd9b4cb64d5a8474d6226be9f11d883ad6a2a2eb46864bbd5b5ec3c4d
|
|
|
AzureDevOpsMigrationTools-Site2
|
7.22 MB |
sha256:ebfc1f4d2b6cc21c06edacac68b96078a08586e0abe871cb2365f46c5a17dc93
|
|