3.818.0 #140
Annotations
3 errors and 11 warnings
|
Publish Manifest
Cloning into '/home/runner/work/vc-module-x-cart/vc-module-x-cart/artifacts/vc-modules'...
|
|
|
|
Publish Manifest
f6c3cc9f..72bb171d HEAD -> master
|
|
Build
/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/Models/CartProduct.cs(23,15): warning S1135: Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135) [/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/VirtoCommerce.XCart.Core.csproj]
|
|
Build
/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/Models/CartAggregateResponseGroup.cs(7,17): warning S2342: Rename this enumeration to match the regular expression: '^([A-Z]{1,3}[a-z0-9]+)*([A-Z]{2})?s$'. (https://rules.sonarsource.com/csharp/RSPEC-2342) [/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/VirtoCommerce.XCart.Core.csproj]
|
|
Build
/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/Models/CartValidationError.cs(33,18): warning S2365: Refactor 'ErrorParameters' into a method, properties should not copy collections. (https://rules.sonarsource.com/csharp/RSPEC-2365) [/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/VirtoCommerce.XCart.Core.csproj]
|
|
Build
/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/CartAggregate.cs(51,29): warning S107: Constructor has 8 parameters, which is greater than the 7 authorized. (https://rules.sonarsource.com/csharp/RSPEC-107) [/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/VirtoCommerce.XCart.Core.csproj]
|
|
Build
/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/Validators/NewCartItemValidator.cs(38,21): warning S1067: Reduce the number of conditional operators (4) used in the expression (maximum allowed 3). (https://rules.sonarsource.com/csharp/RSPEC-1067) [/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/VirtoCommerce.XCart.Core.csproj]
|
|
Build
/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/CartAggregate.cs(176,50): warning S4457: Split this method into two, one handling parameters check and the other handling the asynchronous code. (https://rules.sonarsource.com/csharp/RSPEC-4457) [/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/VirtoCommerce.XCart.Core.csproj]
|
|
Build
/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/CartAggregate.cs(206,50): warning S4457: Split this method into two, one handling parameters check and the other handling the asynchronous code. (https://rules.sonarsource.com/csharp/RSPEC-4457) [/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/VirtoCommerce.XCart.Core.csproj]
|
|
Build
/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/Extensions/TaxExtensions.cs(33,93): warning S109: Assign this magic number '4' to a well-named variable or constant, and use that instead. (https://rules.sonarsource.com/csharp/RSPEC-109) [/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/VirtoCommerce.XCart.Core.csproj]
|
|
Build
/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/Extensions/TaxExtensions.cs(58,91): warning S109: Assign this magic number '4' to a well-named variable or constant, and use that instead. (https://rules.sonarsource.com/csharp/RSPEC-109) [/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/VirtoCommerce.XCart.Core.csproj]
|
|
Build
/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/Extensions/TaxExtensions.cs(89,89): warning S109: Assign this magic number '4' to a well-named variable or constant, and use that instead. (https://rules.sonarsource.com/csharp/RSPEC-109) [/home/runner/work/vc-module-x-cart/vc-module-x-cart/src/VirtoCommerce.XCart.Core/VirtoCommerce.XCart.Core.csproj]
|
|
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/
|
Loading