Skip to content

Refactored Inbox

Refactored Inbox #25

Triggered via pull request March 7, 2024 18:08
Status Success
Total duration 1m 1s
Artifacts

pull-request.yaml

on: pull_request
Pull request
50s
Pull request
Fit to window
Zoom out
Zoom in

Annotations

11 warnings
Pull request
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Pull request: source/TransactionalBox.InboxWorker/Internals/InboxTransportProcessor.cs#L44
The variable 'ex' is declared but never used
Pull request: source/TransactionalBox.Inbox/Internals/InboxProcessor.cs#L44
Possible null reference argument for parameter 'typeArguments' in 'Type Type.MakeGenericType(params Type[] typeArguments)'.
Pull request: source/TransactionalBox.Inbox/Internals/InboxProcessor.cs#L53
Possible null reference assignment.
Pull request: source/TransactionalBox.Inbox/Internals/InboxProcessor.cs#L51
Converting null literal or possible null value to non-nullable type.
Pull request: source/TransactionalBox.Inbox/Internals/InboxProcessor.cs#L51
Dereference of a possibly null reference.
Pull request: source/TransactionalBox.InboxWorker.Kafka/Internals/KafkaInboxWorkerTransport.cs#L15
Async-iterator 'KafkaInboxWorkerTransport.GetMessage(CancellationToken)' has one or more parameters of type 'CancellationToken' but none of them is decorated with the 'EnumeratorCancellation' attribute, so the cancellation token parameter from the generated 'IAsyncEnumerable<>.GetAsyncEnumerator' will be unconsumed
Pull request: source/TransactionalBox.InboxWorker.Kafka/Internals/KafkaInboxWorkerTransport.cs#L15
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.
Pull request: samples/TransactionalBox.Sample.WebApi/InboxMessages/ExampleMessage.cs#L7
Non-nullable property 'Name' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
Pull request: source/TransactionalBox.InboxWorker/Internals/InboxTransportProcessor.cs#L44
The variable 'ex' is declared but never used
Pull request: source/TransactionalBox.Inbox/Internals/InboxProcessor.cs#L44
Possible null reference argument for parameter 'typeArguments' in 'Type Type.MakeGenericType(params Type[] typeArguments)'.