Skip to content
ci

more xml comment fix ups #716

Sign in to view logs

more xml comment fix ups

more xml comment fix ups #716

Triggered via push October 11, 2023 15:27
@neildsouthneildsouth
pushed e5f7474
AI-230
Status Failure
Total duration 5m 13s
Artifacts 2

ci.yml

on: push
Matrix: build
unit-test
3m 5s
unit-test
Matrix: analyze
Publish to GitHub Packages
22s
Publish to GitHub Packages
Official Release to GitHub Packages
0s
Official Release to GitHub Packages
Official Release to NuGet.org
0s
Official Release to NuGet.org
Fit to window
Zoom out
Zoom in

Annotations

21 warnings
unit-test: src/Messaging/Common/MessageConversionException.cs#L17
Remove this empty comment
unit-test: src/Messaging/Common/ServiceException.cs#L21
Update this implementation of 'ISerializable' to conform to the recommended serialization pattern.
unit-test: src/Messaging/IServiceCollectionExtension.cs#L166
"Find" method should be used instead of the "FirstOrDefault" extension method.
unit-test: src/Messaging/IServiceCollectionExtension.cs#L182
"Find" method should be used instead of the "FirstOrDefault" extension method.
unit-test: src/Messaging/Events/EventBase.cs#L129
Loops should be simplified with "LINQ" expressions
unit-test: src/Plugins/RabbitMQ/Factory/RabbitMqConnectionFactory.cs#L41
Refactor 'Connections' into a method, properties should not copy collections.
unit-test: src/Plugins/RabbitMQ/RabbitMQHealthCheck.cs#L49
Remove the unused local variable 'channel'.
unit-test: src/Plugins/RabbitMQ/Factory/RabbitMqConnectionFactory.cs#L59
Use the lambda parameter instead of capturing the argument 'key'
unit-test: src/Plugins/RabbitMQ/Factory/RabbitMqConnectionFactory.cs#L60
Use the lambda parameter instead of capturing the argument 'key'
unit-test: src/Plugins/RabbitMQ/Publisher/RabbitMqMessagePublisherService.cs#L113
'System.NullReferenceException' should not be thrown by user code.
unit-test: src/Messaging/Common/MessageConversionException.cs#L17
Remove this empty comment
unit-test: src/Messaging/Common/ServiceException.cs#L21
Update this implementation of 'ISerializable' to conform to the recommended serialization pattern.
unit-test: src/Messaging/IServiceCollectionExtension.cs#L166
"Find" method should be used instead of the "FirstOrDefault" extension method.
unit-test: src/Messaging/IServiceCollectionExtension.cs#L182
"Find" method should be used instead of the "FirstOrDefault" extension method.
unit-test: src/Messaging/Events/EventBase.cs#L129
Loops should be simplified with "LINQ" expressions
unit-test: src/Plugins/RabbitMQ/Factory/RabbitMqConnectionFactory.cs#L41
Refactor 'Connections' into a method, properties should not copy collections.
unit-test: src/Plugins/RabbitMQ/Subscriber/RabbitMqMessageSubscriberService.cs#L238
Remove the unnecessary Boolean literal(s).
unit-test: src/Plugins/RabbitMQ/Subscriber/RabbitMqMessageSubscriberService.cs#L389
Remove the unused local variable 'testRun'.
unit-test: src/Plugins/RabbitMQ/RabbitMQHealthCheck.cs#L49
Remove the unused local variable 'channel'.
unit-test: src/Plugins/RabbitMQ/Publisher/RabbitMqMessagePublisherService.cs#L113
'System.NullReferenceException' should not be thrown by user code.
analyze (csharp)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Artifacts

Produced during runtime
Name Size
nuget Expired
118 KB
plug-ins Expired
576 KB