Skip to content

VCST-2224: Add support for reply to #256

VCST-2224: Add support for reply to

VCST-2224: Add support for reply to #256

Triggered via pull request November 14, 2024 15:41
Status Success
Total duration 16m 42s
Artifacts 1

module-ci.yml

on: pull_request
module-katalon-tests  /  e2e-tests
13m 20s
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-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/ModuleConstants.cs(27,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-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/VirtoCommerce.NotificationsModule.Core.csproj]
ci
/home/runner/work/vc-module-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/ModuleConstants.cs(27,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-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/VirtoCommerce.NotificationsModule.Core.csproj]
ci
/home/runner/work/vc-module-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/ModuleConstants.cs(27,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-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/VirtoCommerce.NotificationsModule.Core.csproj]
ci
/home/runner/work/vc-module-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/ModuleConstants.cs(46,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-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/VirtoCommerce.NotificationsModule.Core.csproj]
ci
/home/runner/work/vc-module-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/ModuleConstants.cs(36,49): 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-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/VirtoCommerce.NotificationsModule.Core.csproj]
ci
/home/runner/work/vc-module-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/ModuleConstants.cs(36,49): warning S2223: Change the visibility of 'Languages' or make it 'const' or 'readonly'. (https://rules.sonarsource.com/csharp/RSPEC-2223) [/home/runner/work/vc-module-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/VirtoCommerce.NotificationsModule.Core.csproj]
ci
/home/runner/work/vc-module-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/Model/NotificationResponseGroup.cs(6,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-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/VirtoCommerce.NotificationsModule.Core.csproj]
ci
/home/runner/work/vc-module-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/Model/NotificationResponseGroup.cs(8,9): warning S2346: Rename 'Default' to 'None'. (https://rules.sonarsource.com/csharp/RSPEC-2346) [/home/runner/work/vc-module-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/VirtoCommerce.NotificationsModule.Core.csproj]
ci
/home/runner/work/vc-module-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/Model/EmailNotification.cs(54,25): warning S100: Rename property 'BCC' to match pascal case naming rules, consider using 'Bcc'. (https://rules.sonarsource.com/csharp/RSPEC-100) [/home/runner/work/vc-module-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/VirtoCommerce.NotificationsModule.Core.csproj]
ci
/home/runner/work/vc-module-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/Model/EmailNotificationMessage.cs(36,25): warning S100: Rename property 'BCC' to match pascal case naming rules, consider using 'Bcc'. (https://rules.sonarsource.com/csharp/RSPEC-100) [/home/runner/work/vc-module-notification/vc-module-notification/src/VirtoCommerce.NotificationsModule.Core/VirtoCommerce.NotificationsModule.Core.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 Expired
9.04 MB