FullStackHero 10 .NET Starter Kit Release Merge #1152
Draft
Annotations
10 warnings
|
Build:
src/Modules/Identity/Modules.Identity/Features/v1/Groups/AddUsersToGroup/AddUsersToGroupEndpoint.cs#L25
Change 'List<string>' in 'AddUsersRequest.AddUsersRequest(List<string>)' to use 'Collection<T>', 'ReadOnlyCollection<T>' or 'KeyedCollection<K,V>' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1002)
|
|
Build:
src/Modules/Identity/Modules.Identity/Features/v1/Groups/UpdateGroup/UpdateGroupEndpoint.cs#L29
Change 'List<string>' in 'UpdateGroupRequest.UpdateGroupRequest(string, string, bool, List<string>)' to use 'Collection<T>', 'ReadOnlyCollection<T>' or 'KeyedCollection<K,V>' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1002)
|
|
Build:
src/Modules/Identity/Modules.Identity/Features/v1/Groups/UpdateGroup/UpdateGroupEndpoint.cs#L29
Change 'List<string>' in 'UpdateGroupRequest.RoleIds' to use 'Collection<T>', 'ReadOnlyCollection<T>' or 'KeyedCollection<K,V>' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1002)
|
|
Build:
src/Modules/Identity/Modules.Identity/Features/v1/Users/FshUser.cs#L21
Change 'PasswordHistories' to be read-only by removing the property setter (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2227)
|
|
Build:
src/Modules/Identity/Modules.Identity/Features/v1/Groups/Group.cs#L24
Change 'GroupRoles' to be read-only by removing the property setter (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2227)
|
|
Build:
src/Modules/Identity/Modules.Identity/Features/v1/Groups/Group.cs#L25
Change 'UserGroups' to be read-only by removing the property setter (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2227)
|
|
Build:
src/Modules/Identity/Modules.Identity/Features/v1/Groups/AddUsersToGroup/AddUsersToGroupEndpoint.cs#L25
Change 'List<string>' in 'AddUsersRequest.UserIds' to use 'Collection<T>', 'ReadOnlyCollection<T>' or 'KeyedCollection<K,V>' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1002)
|
|
Build:
src/Modules/Multitenancy/Modules.Multitenancy/Features/v1/CreateTenant/CreateTenantCommandValidator.cs#L19
Forward the '_' parameter to the 'ExistsWithNameAsync' method or pass in 'CancellationToken.None' explicitly to indicate intentionally not propagating the token (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2016)
|
|
Build:
src/Modules/Multitenancy/Modules.Multitenancy/Features/v1/CreateTenant/CreateTenantCommandValidator.cs#L14
Forward the '_' parameter to the 'ExistsWithIdAsync' method or pass in 'CancellationToken.None' explicitly to indicate intentionally not propagating the token (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2016)
|
|
Build:
src/BuildingBlocks/Storage/S3/S3StorageService.cs#L124
Logging in a catch clause should pass the caught exception as a parameter. (https://rules.sonarsource.com/csharp/RSPEC-6667)
|
Loading