Skip to content

#273 Wrap IOutboxMigrationService resolution in a scope #859

#273 Wrap IOutboxMigrationService resolution in a scope

#273 Wrap IOutboxMigrationService resolution in a scope #859

Triggered via pull request June 27, 2024 04:07
@EtherZaEtherZa
opened #274
Status Failure
Total duration 7m 18s
Artifacts 1

build.yml

on: pull_request_target
Fit to window
Zoom out
Zoom in

Annotations

1 error and 12 warnings
build
Process completed with exit code 1.
build
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-dotnet@v3, actions/setup-java@v3, actions/upload-artifact@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Deprecation notice: v1, v2, and v3 of the artifact actions
The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "dotnet-results". Please update your workflow to use v4 of the artifact actions. Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/
build: src/SlimMessageBus.Host.Outbox/Interceptors/OutboxSendingTask.cs#L197
Refactor this method to reduce its Cognitive Complexity from 29 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
build: src/SlimMessageBus.Host.Sql.Common/Transactions/SqlTransactionService.cs#L9
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.
build: src/SlimMessageBus.Host.Sql.Common/Transactions/SqlTransactionService.cs#L18
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.
build: src/SlimMessageBus.Host.Outbox/Interceptors/OutboxSendingTask.cs#L197
Refactor this method to reduce its Cognitive Complexity from 29 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
build: src/SlimMessageBus.Host.Sql/SqlTopologyService.cs#L10
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.
build: src/SlimMessageBus.Host.Outbox/Interceptors/OutboxSendingTask.cs#L197
Refactor this method to reduce its Cognitive Complexity from 29 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
build: src/SlimMessageBus.Host.Outbox/Interceptors/OutboxSendingTask.cs#L65
Await CancelAsync instead. (https://rules.sonarsource.com/csharp/RSPEC-6966)
build: src/Samples/Sample.Hybrid.ConsoleApp/Program.cs#L17
Add a 'protected' constructor or the 'static' keyword to the class declaration. (https://rules.sonarsource.com/csharp/RSPEC-1118)
build: src/Samples/Sample.Hybrid.ConsoleApp/Application.cs#L24
Remove the unused local variable 'task'. (https://rules.sonarsource.com/csharp/RSPEC-1481)
build: src/Samples/Sample.DomainEvents.WebApi/Startup.cs#L67
Remove this commented out code. (https://rules.sonarsource.com/csharp/RSPEC-125)

Artifacts

Produced during runtime
Name Size
dotnet-results Expired
38.4 MB