Skip to content
Main Tests & Analysis

Release v0.6.2 #40

Sign in to view logs

Release v0.6.2

Release v0.6.2 #40

Triggered via pull request February 23, 2024 20:46
@Chingling152Chingling152
opened #223
release/v0.6.2
Status Success
Total duration 5m 17s
Artifacts –

branch_main_ci.yml

on: pull_request
Matrix: Tests / Code Tests
Code Analysis  /  Code coverage
2m 43s
Code Analysis / Code coverage
Fit to window
Zoom out
Zoom in

Annotations

46 warnings
Tests / Code Tests (core)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions/setup-dotnet@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Tests / Code Tests (core)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Tests / Code Tests (core): src/api/common/FinancialHub.Common/Results/Errors/ServiceError.cs#L7
#warning: 'it can be changed to an enum or use a different value from https codes'
Tests / Code Tests (core): src/api/core/FinancialHub.Core.Infra.Data/Repositories/TransactionsRepository.cs#L14
#warning: 'This is not a good practice, remove this later'
Tests / Code Tests (core): src/api/core/FinancialHub.Core.Application/Services/TransactionBalanceService.cs#L124
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
Tests / Code Tests (core): src/api/auth/FinancialHub.Auth.Presentation/Controllers/UsersController.cs#L35
Dereference of a possibly null reference.
Tests / Code Tests (core): src/api/auth/FinancialHub.Auth.Presentation/Controllers/UsersController.cs#L41
Possible null reference argument for parameter 'data' in 'ItemResponse<UserModel>.ItemResponse(UserModel data)'.
Tests / Code Tests (core): src/api/auth/FinancialHub.Auth.Presentation/Controllers/SignupController.cs#L34
Dereference of a possibly null reference.
Tests / Code Tests (core): src/api/auth/FinancialHub.Auth.Presentation/Controllers/SignupController.cs#L38
Possible null reference argument for parameter 'data' in 'ItemResponse<UserModel>.ItemResponse(UserModel data)'.
Tests / Code Tests (core): src/api/auth/FinancialHub.Auth.Presentation/Controllers/SigninController.cs#L36
Dereference of a possibly null reference.
Tests / Code Tests (core): src/api/auth/FinancialHub.Auth.Presentation/Controllers/SigninController.cs#L41
Possible null reference argument for parameter 'data' in 'ItemResponse<TokenModel>.ItemResponse(TokenModel data)'.
Tests / Code Tests (core): src/api/auth/FinancialHub.Auth.Presentation/Controllers/UsersController.cs#L64
Dereference of a possibly null reference.
Tests / Code Tests (auth)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions/setup-dotnet@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Tests / Code Tests (auth)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Tests / Code Tests (auth): src/api/common/FinancialHub.Common/Results/Errors/ServiceError.cs#L7
#warning: 'it can be changed to an enum or use a different value from https codes'
Tests / Code Tests (auth): src/api/core/FinancialHub.Core.Infra.Data/Repositories/TransactionsRepository.cs#L14
#warning: 'This is not a good practice, remove this later'
Tests / Code Tests (auth): src/api/core/FinancialHub.Core.Application/Services/TransactionBalanceService.cs#L124
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
Tests / Code Tests (auth): tests/core/FinancialHub.Core.Application.Tests/Services/TransactionBalance/BaseTransactionBalanceTests.cs#L22
'TransactionBalanceService' is obsolete: 'This Service will be removed and the logic will be moved to Balance and Transaction models'
Tests / Code Tests (auth): tests/core/FinancialHub.Core.Application.Tests/Services/TransactionBalance/TransactionBalanceTests.updateamount.cs#L42
'IBalancesService.UpdateAmountAsync(Guid, decimal)' is obsolete: 'This logic will be moved to BalanceModel'
Tests / Code Tests (auth): tests/core/FinancialHub.Core.Application.Tests/Services/TransactionBalance/TransactionBalanceTests.updateamount.cs#L264
'IBalancesService.UpdateAmountAsync(Guid, decimal)' is obsolete: 'This logic will be moved to BalanceModel'
Tests / Code Tests (auth): tests/core/FinancialHub.Core.Application.Tests/Services/TransactionBalance/TransactionBalanceTests.updateamount.cs#L265
'IBalancesService.UpdateAmountAsync(Guid, decimal)' is obsolete: 'This logic will be moved to BalanceModel'
Tests / Code Tests (auth): tests/core/FinancialHub.Core.Application.Tests/Services/TransactionBalance/TransactionBalanceTests.updateamount.cs#L269
'IBalancesService.UpdateAmountAsync(Guid, decimal)' is obsolete: 'This logic will be moved to BalanceModel'
Tests / Code Tests (auth): tests/core/FinancialHub.Core.Application.Tests/Services/TransactionBalance/TransactionBalanceTests.updateamount.cs#L270
'IBalancesService.UpdateAmountAsync(Guid, decimal)' is obsolete: 'This logic will be moved to BalanceModel'
Tests / Code Tests (auth): tests/core/FinancialHub.Core.Application.Tests/Services/TransactionBalance/TransactionBalanceTests.updateamount.cs#L61
'IBalancesService.UpdateAmountAsync(Guid, decimal)' is obsolete: 'This logic will be moved to BalanceModel'
Code Analysis / Code coverage
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions/setup-dotnet@v2, actions/setup-java@v3, actions/cache@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Code Analysis / Code coverage
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/cache@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Code Analysis / Code coverage: src/api/common/FinancialHub.Common/Results/Errors/ServiceError.cs#L7
#warning: 'it can be changed to an enum or use a different value from https codes'
Code Analysis / Code coverage: src/api/core/FinancialHub.Core.Infra.Data/Repositories/TransactionsRepository.cs#L14
#warning: 'This is not a good practice, remove this later'
Code Analysis / Code coverage: src/api/core/FinancialHub.Core.Application/Services/TransactionBalanceService.cs#L124
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
Code Analysis / Code coverage: src/api/auth/FinancialHub.Auth.Presentation/Controllers/SignupController.cs#L34
Dereference of a possibly null reference.
Code Analysis / Code coverage: src/api/auth/FinancialHub.Auth.Presentation/Controllers/SignupController.cs#L38
Possible null reference argument for parameter 'data' in 'ItemResponse<UserModel>.ItemResponse(UserModel data)'.
Code Analysis / Code coverage: src/api/auth/FinancialHub.Auth.Presentation/Controllers/UsersController.cs#L35
Dereference of a possibly null reference.
Code Analysis / Code coverage: src/api/auth/FinancialHub.Auth.Presentation/Controllers/UsersController.cs#L41
Possible null reference argument for parameter 'data' in 'ItemResponse<UserModel>.ItemResponse(UserModel data)'.
Code Analysis / Code coverage: src/api/auth/FinancialHub.Auth.Presentation/Controllers/SigninController.cs#L36
Dereference of a possibly null reference.
Code Analysis / Code coverage: src/api/auth/FinancialHub.Auth.Presentation/Controllers/SigninController.cs#L41
Possible null reference argument for parameter 'data' in 'ItemResponse<TokenModel>.ItemResponse(TokenModel data)'.
Code Analysis / Code coverage: src/api/auth/FinancialHub.Auth.Presentation/Controllers/UsersController.cs#L64
Dereference of a possibly null reference.
Code Analysis / Code coverage: src/api/common/FinancialHub.Common/Results/Errors/ServiceError.cs#L7
#warning: 'it can be changed to an enum or use a different value from https codes'
Code Analysis / Code coverage: src/api/core/FinancialHub.Core.Infra.Data/Repositories/TransactionsRepository.cs#L14
#warning: 'This is not a good practice, remove this later'
Code Analysis / Code coverage: src/api/core/FinancialHub.Core.Application/Services/TransactionBalanceService.cs#L124
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
Code Analysis / Code coverage: src/api/core/FinancialHub.Core.WebApi/appsettings.Testing.json#L10
"pwd" detected here, make sure this is not a hard-coded credential.
Code Analysis / Code coverage: src/api/core/FinancialHub.Core.WebApi/Controllers/TransactionsController.cs#L56
Do not forget to remove this deprecated code someday.
Code Analysis / Code coverage: src/api/auth/FinancialHub.Auth.Presentation/Controllers/SigninController.cs#L36
Dereference of a possibly null reference.
Code Analysis / Code coverage: src/api/auth/FinancialHub.Auth.Presentation/Controllers/SigninController.cs#L41
Possible null reference argument for parameter 'data' in 'ItemResponse<TokenModel>.ItemResponse(TokenModel data)'.
Code Analysis / Code coverage: src/api/auth/FinancialHub.Auth.Presentation/Controllers/SignupController.cs#L34
Dereference of a possibly null reference.
Code Analysis / Code coverage: src/api/auth/FinancialHub.Auth.Presentation/Controllers/SignupController.cs#L38
Possible null reference argument for parameter 'data' in 'ItemResponse<UserModel>.ItemResponse(UserModel data)'.
Code Analysis / Code coverage: src/api/auth/FinancialHub.Auth.Presentation/Controllers/UsersController.cs#L35
Dereference of a possibly null reference.