18 new issues (0 max.) of at least minor severity.
Here is an overview of what got changed by this pull request:
Issues
======
- Added 18
Complexity increasing per file
==============================
- src/KafkaFlow.IntegrationTests/OpenTelemetryTests.cs 1
- src/KafkaFlow.Abstractions/MessageEventExceptionContext.cs 1
- src/KafkaFlow.OpenTelemetry/OpenTelemetryConsumerObserver.cs 5
- src/KafkaFlow.IntegrationTests/GlobalEventsTest.cs 1
- src/KafkaFlow.OpenTelemetry/KafkaFlowActivitySourceHelper.cs 2
- src/KafkaFlow.OpenTelemetry/OpenTelemetryProducerObserver.cs 7
- src/KafkaFlow/GlobalEvents.cs 1
- src/KafkaFlow.Abstractions/MessageEventContext.cs 1
- src/KafkaFlow/EventSubscription.cs 1
- src/KafkaFlow/Event.cs 3
- src/KafkaFlow.OpenTelemetry/ExtensionMethods.cs 1
Clones added
============
- src/KafkaFlow.IntegrationTests/OpenTelemetryTests.cs 1
- src/KafkaFlow.IntegrationTests/GlobalEventsTest.cs 2
- src/KafkaFlow.IntegrationTests/Core/Bootstrapper.cs 1
See the complete overview on Codacy
Annotations
Check notice on line 49 in src/KafkaFlow.IntegrationTests/GlobalEventsTest.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.IntegrationTests/GlobalEventsTest.cs#L49
Remove the unused local variable 'consumer'.
Check warning on line 42 in src/KafkaFlow.IntegrationTests/OpenTelemetryTests.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.IntegrationTests/OpenTelemetryTests.cs#L42
Remove this commented out code.
Check failure on line 134 in src/KafkaFlow.IntegrationTests/OpenTelemetryTests.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.IntegrationTests/OpenTelemetryTests.cs#L134
Add a nested comment explaining why this method is empty, throw a 'NotSupportedException' or complete the implementation.
Check notice on line 13 in src/KafkaFlow.OpenTelemetry/KafkaFlowActivitySourceHelper.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.OpenTelemetry/KafkaFlowActivitySourceHelper.cs#L13
Replace this 'static readonly' declaration with 'const'.
Check notice on line 14 in src/KafkaFlow.OpenTelemetry/KafkaFlowActivitySourceHelper.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.OpenTelemetry/KafkaFlowActivitySourceHelper.cs#L14
Replace this 'static readonly' declaration with 'const'.
Check notice on line 15 in src/KafkaFlow.OpenTelemetry/KafkaFlowActivitySourceHelper.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.OpenTelemetry/KafkaFlowActivitySourceHelper.cs#L15
Replace this 'static readonly' declaration with 'const'.
Check notice on line 16 in src/KafkaFlow.OpenTelemetry/KafkaFlowActivitySourceHelper.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.OpenTelemetry/KafkaFlowActivitySourceHelper.cs#L16
Replace this 'static readonly' declaration with 'const'.
Check notice on line 14 in src/KafkaFlow.OpenTelemetry/OpenTelemetryConsumerObserver.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.OpenTelemetry/OpenTelemetryConsumerObserver.cs#L14
Replace this 'static readonly' declaration with 'const'.
Check warning on line 20 in src/KafkaFlow.OpenTelemetry/OpenTelemetryConsumerObserver.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.OpenTelemetry/OpenTelemetryConsumerObserver.cs#L20
'context' is null on at least one execution path.
Check warning on line 31 in src/KafkaFlow.OpenTelemetry/OpenTelemetryConsumerObserver.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.OpenTelemetry/OpenTelemetryConsumerObserver.cs#L31
'context' is null on at least one execution path.
Check notice on line 40 in src/KafkaFlow.OpenTelemetry/OpenTelemetryConsumerObserver.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.OpenTelemetry/OpenTelemetryConsumerObserver.cs#L40
Handle the exception or explain in a comment why it can be ignored.
Check warning on line 41 in src/KafkaFlow.OpenTelemetry/OpenTelemetryConsumerObserver.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.OpenTelemetry/OpenTelemetryConsumerObserver.cs#L41
Either remove or fill this block of code.
Check notice on line 14 in src/KafkaFlow.OpenTelemetry/OpenTelemetryProducerObserver.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.OpenTelemetry/OpenTelemetryProducerObserver.cs#L14
Replace this 'static readonly' declaration with 'const'.
Check warning on line 20 in src/KafkaFlow.OpenTelemetry/OpenTelemetryProducerObserver.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.OpenTelemetry/OpenTelemetryProducerObserver.cs#L20
'context' is null on at least one execution path.
Check warning on line 27 in src/KafkaFlow.OpenTelemetry/OpenTelemetryProducerObserver.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.OpenTelemetry/OpenTelemetryProducerObserver.cs#L27
'context' is null on at least one execution path.
Check notice on line 54 in src/KafkaFlow.OpenTelemetry/OpenTelemetryProducerObserver.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.OpenTelemetry/OpenTelemetryProducerObserver.cs#L54
Handle the exception or explain in a comment why it can be ignored.
Check warning on line 55 in src/KafkaFlow.OpenTelemetry/OpenTelemetryProducerObserver.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.OpenTelemetry/OpenTelemetryProducerObserver.cs#L55
Either remove or fill this block of code.
Check warning on line 100 in src/KafkaFlow.OpenTelemetry/OpenTelemetryProducerObserver.cs
codacy-production / Codacy Static Code Analysis
src/KafkaFlow.OpenTelemetry/OpenTelemetryProducerObserver.cs#L100
'context' is null on at least one execution path.