Adding documentation & comments #4
fkitsantasAnnotations
15 warnings
|
build-and-test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-dotnet@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
build-and-test:
LondonStockExchange.UnitTests/Controllers/StocksControllerTests.cs#L77
Converting null literal or possible null value to non-nullable type.
|
|
build-and-test:
LondonStockExchange.UnitTests/Controllers/StocksControllerTests.cs#L77
Argument of type 'ISetup<IStockService, Task<StockDTO>>' cannot be used for parameter 'mock' of type 'IReturns<IStockService, Task<StockDTO?>>' in 'IReturnsResult<IStockService> ReturnsExtensions.ReturnsAsync<IStockService, StockDTO?>(IReturns<IStockService, Task<StockDTO?>> mock, StockDTO? value)' due to differences in the nullability of reference types.
|
|
build-and-test:
LondonStockExchange.UnitTests/Controllers/StocksControllerTests.cs#L77
Converting null literal or possible null value to non-nullable type.
|
|
build-and-test:
LondonStockExchange.UnitTests/Controllers/StocksControllerTests.cs#L77
Argument of type 'ISetup<IStockService, Task<StockDTO>>' cannot be used for parameter 'mock' of type 'IReturns<IStockService, Task<StockDTO?>>' in 'IReturnsResult<IStockService> ReturnsExtensions.ReturnsAsync<IStockService, StockDTO?>(IReturns<IStockService, Task<StockDTO?>> mock, StockDTO? value)' due to differences in the nullability of reference types.
|
|
build-and-test:
LondonStockExchange.API/Services/StockService.cs#L48
Possible null reference return.
|
|
build-and-test:
LondonStockExchange.API/DTOs/TransactionDTO.cs#L23
Non-nullable property 'TickerSymbol' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
|
build-and-test:
LondonStockExchange.API/DTOs/TradeResult.cs#L29
Non-nullable property 'Message' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
|
build-and-test:
LondonStockExchange.API/DTOs/TradeResult.cs#L38
Non-nullable property 'UpdatedStock' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
|
build-and-test:
LondonStockExchange.API/DTOs/TradeDTO.cs#L21
Non-nullable property 'TickerSymbol' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
|
build-and-test:
LondonStockExchange.API/DTOs/StockDTO.cs#L21
Non-nullable property 'TickerSymbol' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
|
build-and-test:
LondonStockExchange.API/Services/StockService.cs#L77
Possible null reference return.
|
|
build-and-test:
LondonStockExchange.API/Repositories/StockRepository.cs#L44
Possible null reference return.
|
|
build-and-test:
LondonStockExchange.API/Repositories/StockRepository.cs#L54
Possible null reference return.
|
|
build-and-test:
LondonStockExchange.API/Repositories/BrokerRepository.cs#L43
Possible null reference return.
|