Skip to content
Module CI

VCST-2523: Adding images to reviews #185

Sign in to view logs

VCST-2523: Adding images to reviews

VCST-2523: Adding images to reviews #185

Triggered via pull request January 22, 2025 12:33
@LjutyjLjutyj
synchronize #70
feat/VCST-2523
Status Success
Total duration 16m 36s
Artifacts 1

module-ci.yml

on: pull_request
ci
2m 28s
ci
module-katalon-tests  /  e2e-tests
13m 50s
module-katalon-tests / e2e-tests
Matrix: deploy-cloud / Deploy
Waiting for pending jobs
Fit to window
Zoom out
Zoom in

Annotations

11 warnings
ci
/home/runner/work/vc-module-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.Core/ModuleConstants.cs(25,31): warning S2386: Use an immutable collection or reduce the accessibility of the public static field 'AllPermissions'. (https://rules.sonarsource.com/csharp/RSPEC-2386) [/home/runner/work/vc-module-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.Core/VirtoCommerce.CustomerReviews.Core.csproj]
ci
/home/runner/work/vc-module-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.Core/ModuleConstants.cs(25,40): warning S1104: Make this field 'private' and encapsulate it in a 'public' property. (https://rules.sonarsource.com/csharp/RSPEC-1104) [/home/runner/work/vc-module-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.Core/VirtoCommerce.CustomerReviews.Core.csproj]
ci
/home/runner/work/vc-module-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.Core/ModuleConstants.cs(25,40): warning S2223: Change the visibility of 'AllPermissions' or make it 'const' or 'readonly'. (https://rules.sonarsource.com/csharp/RSPEC-2223) [/home/runner/work/vc-module-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.Core/VirtoCommerce.CustomerReviews.Core.csproj]
ci
/home/runner/work/vc-module-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.Core/ModuleConstants.cs(116,62): warning S3218: Rename this property to not shadow the outer class' member with the same name. (https://rules.sonarsource.com/csharp/RSPEC-3218) [/home/runner/work/vc-module-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.Core/VirtoCommerce.CustomerReviews.Core.csproj]
ci
/home/runner/work/vc-module-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.ExperienceApi/Middleware/EvalVendorRatingMiddleware.cs(23,31): 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-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.ExperienceApi/VirtoCommerce.CustomerReviews.ExperienceApi.csproj]
ci
/home/runner/work/vc-module-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.ExperienceApi/Middleware/EvalProductVendorRatingMiddleware.cs(27,31): 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-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.ExperienceApi/VirtoCommerce.CustomerReviews.ExperienceApi.csproj]
ci
/home/runner/work/vc-module-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.ExperienceApi/Middleware/EvalProductRatingMiddleware.cs(26,31): 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-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.ExperienceApi/VirtoCommerce.CustomerReviews.ExperienceApi.csproj]
ci
/home/runner/work/vc-module-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.ExperienceApi/Commands/CreateReviewCommandHandler.cs(103,25): 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-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.ExperienceApi/VirtoCommerce.CustomerReviews.ExperienceApi.csproj]
ci
/home/runner/work/vc-module-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.Data/Repositories/CustomerReviewRepository.cs(102,21): warning S4136: All 'Delete' method overloads should be adjacent. (https://rules.sonarsource.com/csharp/RSPEC-4136) [/home/runner/work/vc-module-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.Data/VirtoCommerce.CustomerReviews.Data.csproj]
ci
/home/runner/work/vc-module-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.Data/Repositories/ICustomerReviewRepository.cs(31,14): warning S4136: All 'Delete' method overloads should be adjacent. (https://rules.sonarsource.com/csharp/RSPEC-4136) [/home/runner/work/vc-module-customer-review/vc-module-customer-review/src/VirtoCommerce.CustomerReviews.Data/VirtoCommerce.CustomerReviews.Data.csproj]
ci
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/

Artifacts

Produced during runtime
Name Size
reports
8.61 MB