Skip to content

VCST-509: optional modules support #929

VCST-509: optional modules support

VCST-509: optional modules support #929

Triggered via pull request May 9, 2024 13:21
Status Success
Total duration 4m 32s
Artifacts

pr-ci.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

20 warnings
build / build-artifacts
npm WARN deprecated angular-cookies@1.6.10: For the actively supported Angular, see https://www.npmjs.com/package/@angular/core. AngularJS support has officially ended. For extended AngularJS support options, see https://goo.gle/angularjs-path-forward.
build / build-artifacts
npm WARN deprecated urix@0.1.0: Please see https://github.com/lydell/urix#deprecated
build / build-artifacts
npm WARN deprecated resolve-url@0.2.1: https://github.com/lydell/resolve-url#deprecated
build / build-artifacts
npm WARN deprecated angular-cookies@1.8.3: For the actively supported Angular, see https://www.npmjs.com/package/@angular/core. AngularJS support has officially ended. For extended AngularJS support options, see https://goo.gle/angularjs-path-forward.
build / build-artifacts
npm WARN deprecated source-map-resolve@0.5.3: See https://github.com/lydell/source-map-resolve#deprecated
build / build-artifacts
npm WARN deprecated har-validator@5.1.5: this library is no longer supported
build / build-artifacts
npm WARN deprecated source-map-url@0.4.1: See https://github.com/lydell/source-map-url#deprecated
build / build-artifacts
npm WARN deprecated querystring@0.2.0: The querystring API is considered Legacy. new code should use the URLSearchParams API instead.
build / build-artifacts
npm WARN deprecated angular-resource@1.8.3: For the actively supported Angular, see https://www.npmjs.com/package/@angular/core. AngularJS support has officially ended. For extended AngularJS support options, see https://goo.gle/angularjs-path-forward.
build / build-artifacts
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.
test / test-and-sonar
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Web/VirtoCommerce.Platform.Web.csproj : warning NU1901: Package 'Microsoft.Identity.Client' 4.58.0 has a known low severity vulnerability, https://github.com/advisories/GHSA-x674-v45j-fwxw [/home/runner/work/vc-platform/vc-platform/VirtoCommerce.Platform.sln]
test / test-and-sonar
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/Notifications/DefaultEmailSender.cs(5,39): warning CS0618: 'IEmailSender' is obsolete: 'Use notification module instead' [/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/VirtoCommerce.Platform.Core.csproj]
test / test-and-sonar
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/Security/ApplicationUser.cs(107,13): warning CS0618: 'ApplicationUser.UserState' is obsolete: 'Left due to compatibility issues. Will be removed. Instead of, use properties: EmailConfirmed, LockoutEnd.' [/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/VirtoCommerce.Platform.Core.csproj]
test / test-and-sonar
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/Security/ApplicationUser.cs(107,32): warning CS0618: 'ApplicationUser.UserState' is obsolete: 'Left due to compatibility issues. Will be removed. Instead of, use properties: EmailConfirmed, LockoutEnd.' [/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/VirtoCommerce.Platform.Core.csproj]
test / test-and-sonar
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/Modularity/ModuleCatalog.cs(477,1): warning S103: Split this 365 characters long line (which is greater than 300 authorized). (https://rules.sonarsource.com/csharp/RSPEC-103) [/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/VirtoCommerce.Platform.Core.csproj]
test / test-and-sonar
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/Modularity/ModuleCatalog.cs(510,23): warning S3260: Private classes which are not derived in the current assembly should be marked as 'sealed'. (https://rules.sonarsource.com/csharp/RSPEC-3260) [/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/VirtoCommerce.Platform.Core.csproj]
test / test-and-sonar
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/Modularity/IOptionalDependency.cs(3,38): warning S3246: Add the 'out' keyword to parameter 'T' to make it 'covariant'. (https://rules.sonarsource.com/csharp/RSPEC-3246) [/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/VirtoCommerce.Platform.Core.csproj]
test / test-and-sonar
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/Modularity/ManifestModuleInfo.cs(20,37): warning S1144: Remove the unused private setter 'set_Optional'. (https://rules.sonarsource.com/csharp/RSPEC-1144) [/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/VirtoCommerce.Platform.Core.csproj]
test / test-and-sonar
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/Modularity/ModuleCatalog.cs(50,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.Core/VirtoCommerce.Platform.Core.csproj]
test / test-and-sonar
/home/runner/work/vc-platform/vc-platform/src/VirtoCommerce.Platform.Core/Modularity/ModuleCatalog.cs(229,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.Core/VirtoCommerce.Platform.Core.csproj]