Refactor serilog initialization and add module initialization logging #1244
Annotations
10 warnings
|
Build
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/Logger/ConsoleLog.cs(10,13): warning S106: Remove this logging statement. (https://rules.sonarsource.com/csharp/RSPEC-106) [/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/VirtoCommerce.Platform.Core.csproj]
|
|
Build
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/Logger/ConsoleLog.cs(15,13): warning S106: Remove this logging statement. (https://rules.sonarsource.com/csharp/RSPEC-106) [/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/VirtoCommerce.Platform.Core.csproj]
|
|
Build
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Modules/ModuleManager.cs(106,85): warning S927: Rename parameter 'applicationBuilder' to 'appBuilder' to match the interface declaration. (https://rules.sonarsource.com/csharp/RSPEC-927) [/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Modules/VirtoCommerce.Platform.Modules.csproj]
|
|
Build
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Modules/ModuleManager.cs(108,13): warning S121: Add curly braces around the nested statement(s) in this 'if' block. (https://rules.sonarsource.com/csharp/RSPEC-121) [/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Modules/VirtoCommerce.Platform.Modules.csproj]
|
|
Build
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Modules/ModuleManager.cs(96,17): warning S2589: Change this condition so that it does not always evaluate to 'False'. (https://rules.sonarsource.com/csharp/RSPEC-2589) [/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Modules/VirtoCommerce.Platform.Modules.csproj]
|
|
Build
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Modules/ModuleInitializer.cs(94,79): warning S927: Rename parameter 'appBuilder' to 'serviceProvider' to match the interface declaration. (https://rules.sonarsource.com/csharp/RSPEC-927) [/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Modules/VirtoCommerce.Platform.Modules.csproj]
|
|
Build
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Modules/ModuleManager.cs(292,13): warning S121: Add curly braces around the nested statement(s) in this 'if' block. (https://rules.sonarsource.com/csharp/RSPEC-121) [/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Modules/VirtoCommerce.Platform.Modules.csproj]
|
|
Build
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Modules/ModuleManager.cs(322,54): warning S3267: Loops should be simplified using the "Where" LINQ method (https://rules.sonarsource.com/csharp/RSPEC-3267) [/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Modules/VirtoCommerce.Platform.Modules.csproj]
|
|
Build
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Data.MySql/Migrations/Data/20221128135153_Initial.cs(10,33): warning S138: This method 'Up' has 253 lines, which is greater than the 80 lines authorized. Split it into smaller methods. (https://rules.sonarsource.com/csharp/RSPEC-138) [/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Data.MySql/VirtoCommerce.Platform.Data.MySql.csproj]
|
|
Build
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Data.MySql/Migrations/Security/20221128141556_Initial.cs(10,33): warning S138: This method 'Up' has 520 lines, which is greater than the 80 lines authorized. Split it into smaller methods. (https://rules.sonarsource.com/csharp/RSPEC-138) [/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Data.MySql/VirtoCommerce.Platform.Data.MySql.csproj]
|
Loading