Refactor serilog initialization and add module initialization logging #1296
Annotations
10 errors and 11 warnings
|
Packaging
npm warn deprecated urix@0.1.0: Please see https://github.com/lydell/urix#deprecated
|
|
Packaging
npm warn deprecated source-map-url@0.4.1: See https://github.com/lydell/source-map-url#deprecated
|
|
Packaging
npm warn deprecated resolve-url@0.2.1: https://github.com/lydell/resolve-url#deprecated
|
|
Packaging
npm warn deprecated source-map-resolve@0.5.3: See https://github.com/lydell/source-map-resolve#deprecated
|
|
Packaging
npm warn deprecated querystring@0.2.0: The querystring API is considered Legacy. new code should use the URLSearchParams API instead.
|
|
Packaging
npm warn deprecated uuid@3.4.0: Please upgrade to version 7 or higher. Older versions may use Math.random() in certain circumstances, which is known to be problematic. See https://v8.dev/blog/math-random for details.
|
|
Packaging
npm warn deprecated har-validator@5.1.5: this library is no longer supported
|
|
Packaging
npm warn deprecated request@2.88.2: request has been deprecated, see https://github.com/request/request/issues/3142
|
|
Packaging
npm warn deprecated angular-translate-storage-local@2.19.1: Package no longer supported. Contact Support at https://www.npmjs.com/support for more info.
|
|
Packaging
npm warn deprecated angular-translate-loader-url@2.19.1: Package no longer supported. Contact Support at https://www.npmjs.com/support for more info.
|
|
Get changelog
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
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.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.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.PostgreSql/Migrations/Data/20221128134856_Initial.cs(10,33): warning S138: This method 'Up' has 197 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.PostgreSql/VirtoCommerce.Platform.Data.PostgreSql.csproj]
|
|
Build
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Data.PostgreSql/Migrations/Security/20221128141520_Initial.cs(10,33): warning S138: This method 'Up' has 410 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.PostgreSql/VirtoCommerce.Platform.Data.PostgreSql.csproj]
|
Loading