118 code review feedback luke parker #83
Annotations
8 warnings
Src/Application/Customers/EventHandlers/CustomerCreatedHandler.cs#L10
In member Task CustomerCreatedHandler.Handle(CustomerCreatedEvent evt, CancellationToken cancellationToken), change parameter name evt to notification in order to match the identifier as it has been declared in Task INotificationHandler<CustomerCreatedEvent>.Handle(CustomerCreatedEvent notification, CancellationToken cancellationToken) (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1725)
|
Tests/Common/Fixtures/WebUITestFactory.cs#L95
Non-nullable property 'tokenType' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
Tests/Common/Fixtures/WebUITestFactory.cs#L96
Non-nullable property 'accessToken' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
Tests/Common/Fixtures/WebUITestFactory.cs#L98
Non-nullable property 'refreshToken' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
Src/Application/Customers/EventHandlers/CustomerCreatedHandler.cs#L10
In member Task CustomerCreatedHandler.Handle(CustomerCreatedEvent evt, CancellationToken cancellationToken), change parameter name evt to notification in order to match the identifier as it has been declared in Task INotificationHandler<CustomerCreatedEvent>.Handle(CustomerCreatedEvent notification, CancellationToken cancellationToken) (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1725)
|
Tests/Common/Fixtures/WebUITestFactory.cs#L95
Non-nullable property 'tokenType' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
Tests/Common/Fixtures/WebUITestFactory.cs#L96
Non-nullable property 'accessToken' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
Tests/Common/Fixtures/WebUITestFactory.cs#L98
Non-nullable property 'refreshToken' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
The logs for this run have expired and are no longer available.
Loading