Small improvements after upgrading to .NET 9, React 19, and Lingui v5 #605
tjementumaccount-management.yml
on: pull_request
Build and Test
2m 52s
Code Style and Linting
3m 29s
Deploy API
/
Production
Deploy Workers
/
Production
Annotations
12 warnings
|
Build and Test
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|
|
Build and Test:
application/AppGateway/Program.cs#L47
Refactor your code not to use hardcoded absolute paths or URIs. (https://rules.sonarsource.com/csharp/RSPEC-1075)
|
|
Build and Test:
application/account-management/Core/Features/Authentication/Commands/RefreshAuthenticationTokens.cs#L78
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
|
|
Build and Test:
application/account-management/Core/Features/Authentication/Commands/StartLogin.cs#L56
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
|
|
Build and Test:
application/account-management/Core/Database/DatabaseMigrations.cs#L17
Define a constant instead of using this literal 'datetimeoffset' 10 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
Build and Test:
application/account-management/Core/Database/DatabaseMigrations.cs#L19
Define a constant instead of using this literal 'varchar(30)' 4 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
Build and Test:
application/account-management/Core/Features/Authentication/Commands/RefreshAuthenticationTokens.cs#L34
Define a constant instead of using this literal 'Invalid refresh token' 6 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
Build and Test:
application/account-management/Core/Features/Users/Shared/AvatarUpdater.cs#L34
Make sure this weak hash algorithm is not used in a sensitive context here. (https://rules.sonarsource.com/csharp/RSPEC-4790)
|
|
Build and Test:
application/account-management/Core/Features/Authentication/Commands/CompleteLogin.cs#L21
Constructor has 8 parameters, which is greater than the 7 authorized. (https://rules.sonarsource.com/csharp/RSPEC-107)
|
|
Build and Test:
application/account-management/Core/Features/Authentication/Commands/CompleteLogin.cs#L82
Merge this if statement with the enclosing one. (https://rules.sonarsource.com/csharp/RSPEC-1066)
|
|
Build and Test:
application/account-management/Core/Features/Users/Domain/UserRepository.cs#L73
Refactor this method to reduce its Cognitive Complexity from 16 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
|
|
Code Style and Linting
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|