Skip to content

Fix sonar warnings #1126

Fix sonar warnings

Fix sonar warnings #1126

Triggered via pull request September 19, 2024 20:02
@dundichdundich
synchronize #307
Status Success
Total duration 10m 2s
Artifacts 2

build.yml

on: pull_request_target
Fit to window
Zoom out
Zoom in

Annotations

11 warnings
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/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/MessageBusBase.cs#L666
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
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.Nats/NatsSubjectConsumer.cs#L36
Avoid awaiting or returning a Task representing work that was not started within your context as that can lead to deadlocks.
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/MessageBusBase.cs#L666
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)

Artifacts

Produced during runtime
Name Size
nuget-packages
3.71 MB
test-results
218 KB