Skip to content

Fix sonar warnings #1124

Fix sonar warnings

Fix sonar warnings #1124

Triggered via pull request September 19, 2024 08:53
@dundichdundich
synchronize #307
Status Failure
Total duration 11m 29s
Artifacts 1

build.yml

on: pull_request_target
Fit to window
Zoom out
Zoom in

Annotations

1 error and 11 warnings
build
Process completed with exit code 1.
build
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-dotnet@v3, actions/setup-java@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
build: src/SlimMessageBus.Host/MessageBusBase.cs#L666
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
build: src/SlimMessageBus.Host/Helpers/ReflectionUtils.cs#L19
Update this method so that its implementation is not identical to 'GenerateGetterExpr'. (https://rules.sonarsource.com/csharp/RSPEC-4144)
build: src/SlimMessageBus.Host/Helpers/ReflectionUtils.cs#L10
Define a constant instead of using this literal 'instance' 5 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
build: src/SlimMessageBus.Host/MessageBusBase.cs#L575
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
build: src/SlimMessageBus.Host/MessageBusBase.cs#L559
Method has 11 parameters, which is greater than the 7 authorized. (https://rules.sonarsource.com/csharp/RSPEC-107)
build: src/SlimMessageBus.Host.Outbox/Interceptors/OutboxForwardingPublishInterceptor.cs#L7
Remove this empty class, write its code or make it an "interface". (https://rules.sonarsource.com/csharp/RSPEC-2094)
build: src/SlimMessageBus.Host.Outbox/Services/OutboxSendingTask.cs#L252
Refactor this method to reduce its Cognitive Complexity from 25 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
build: src/SlimMessageBus.Host.Sql.Common/CommonSqlMigrationService.cs#L114
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
build: src/SlimMessageBus.Host.Kafka/Consumer/KafkaGroupConsumer.cs#L65
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
build: src/SlimMessageBus.Host.Kafka/Consumer/KafkaPartitionConsumer.cs#L40
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)

Artifacts

Produced during runtime
Name Size
test-results
197 KB